Skip to content

fix: pass hiddenStatuses to second renderOverlays call (build fix)#1731

Merged
Hugo0 merged 3 commits intomainfrom
fix/graph-legend-build-fix
Mar 10, 2026
Merged

fix: pass hiddenStatuses to second renderOverlays call (build fix)#1731
Hugo0 merged 3 commits intomainfrom
fix/graph-legend-build-fix

Conversation

@Hugo0
Copy link
Contributor

@Hugo0 Hugo0 commented Mar 10, 2026

Summary

  • The minimal/mobile mode renderOverlays call at line 2548 was missing hiddenStatuses and setHiddenStatuses props added in 2d05199, causing a TypeScript build failure.

Test plan

  • Verify build passes
  • Verify full-graph legend toggles work

Hugo0 added 2 commits March 10, 2026 17:18
The minimal/mobile mode renderOverlays call was missing the new
hiddenStatuses and setHiddenStatuses props, causing a type error.
@vercel
Copy link

vercel bot commented Mar 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
peanut-wallet Ready Ready Preview, Comment Mar 10, 2026 5:34pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: fa7113a4-23df-43b6-897a-6c50a7e513c8

📥 Commits

Reviewing files that changed from the base of the PR and between 31feb5e and 3d34c13.

📒 Files selected for processing (3)
  • CONTRIBUTING.md
  • docs/archive/SOP.md
  • src/constants/chains.consts.ts
✅ Files skipped from review due to trivial changes (1)
  • src/constants/chains.consts.ts

Walkthrough

Adds runtime exposure of hiddenStatuses and its setter to overlays in the InvitesGraph render path, and makes small documentation updates to CONTRIBUTING.md and top-of-file guidance in chains.consts.ts.

Changes

Cohort / File(s) Summary
Graph Overlay Configuration
src/components/Global/InvitesGraph/index.tsx
Now passes hiddenStatuses and setHiddenStatuses to the renderOverlays callback in both minimal and full rendering branches, enabling overlays to read and toggle status visibility at runtime.
Contributor Workflow Docs
CONTRIBUTING.md
Replaces "Parallel Work" header with "Git Workflow", adds rules to never commit/push to main and to always use feature branches; narrows docs exceptions to exclude only README.md and CONTRIBUTING.md.
Chains Constants Comment
src/constants/chains.consts.ts
Expands top-of-file comment with detailed instructions for adding new chains (prefer wagmi imports, otherwise create custom chain and append to supportedPeanutChains). No code behavior changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding missing hiddenStatuses props to a renderOverlays call to fix a TypeScript build failure.
Description check ✅ Passed The description is directly related to the changeset, explaining the missing props issue and providing a test plan relevant to the changes.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/graph-legend-build-fix

Comment @coderabbitai help to get the list of available commands and usage tips.

Add top-level rules: never push directly to main, always use feature
branches and PRs, wait for CI before merging.
@Hugo0 Hugo0 merged commit 7ff3a88 into main Mar 10, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant