Enable maintenance banner

[Zishan-7]

No description provided.

[vercel]

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

Project	Deployment	Preview	Comments	Updated (UTC)
peanut-wallet	Error			Oct 20, 2025 8:50am

💡 Enable Vercel Agent with $100 free credit for automated AI reviews

[coderabbitai]

Walkthrough

Two config flags in underMaintenanceConfig were enabled (enableFullMaintenance, enableMaintenanceBanner), and a Listbox value binding was adjusted to allow undefined in Select component rendering.

Changes

Cohort / File(s)	Summary
Configuration flag enablement src/config/underMaintenance.config.ts	Flipped enableFullMaintenance and enableMaintenanceBanner from false to true.
Select component minor binding fix src/components/Global/Select/index.tsx	Adjusted Listbox value binding to pass value or undefined to handle null/undefined without type/conflict issues.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix: under maintenance setup, now only need to set two flags #1280: Introduces and wires the underMaintenanceConfig flags that this PR activates.
• chore: enable maintenance on all pages  #1233: Implements middleware and layout/banner rendering controlled by the maintenance flags.
• chore: remove maintenance banner #1295: Modifies the same config flag(s) and may overlap or conflict.

Suggested reviewers

• kushagrasarathe
• jjramirezn
• Hugo0

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title "Enable maintenance banner" directly references a real and significant change in the pull request—the enableMaintenanceBanner configuration flag being set to true in underMaintenance.config.ts. However, the changeset also includes enabling full maintenance mode (enableFullMaintenance set to true) in the same configuration file, plus an unrelated change to the Select component's Listbox value binding. The title accurately captures one key aspect of the primary change but does not encompass the full scope of the modifications.
Description Check	✅ Passed	No pull request description was provided by the author. The description field is empty and contains no information about the changeset. However, according to the evaluation criteria, this is an exceptionally lenient check designed to pass as long as the description is not completely off-topic. An empty description, while lacking detail, is not off-topic in nature and therefore meets the minimum threshold for this particular check.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/enable-maintainence-banner

---

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

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between df9d3aa and 7fe581b.

📒 Files selected for processing (1)

• src/config/underMaintenance.config.ts (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Deploy-Preview

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

src/components/Global/Select/index.tsx (1)

47-47: Consider using nullish coalescing for more precise null handling.

The change correctly converts null to undefined for Listbox compatibility. However, using the nullish coalescing operator (??) instead of logical OR (||) would be more semantically precise and clearer in intent.

Apply this diff:

-            <Listbox value={value || undefined} onChange={onChange}>
+            <Listbox value={value ?? undefined} onChange={onChange}>

Alternatively, consider updating the type definition to explicitly accept undefined:

-    value: SelectItem | null
+    value: SelectItem | null | undefined

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7fe581b and 0ffc115.

📒 Files selected for processing (1)

• src/components/Global/Select/index.tsx (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Deploy-Preview

[Zishan-7]

Not needed anymore