Skip to content

chore: remove maintenance banner#1295

Merged
jjramirezn merged 1 commit intopeanut-walletfrom
fix/maintainance-setup
Oct 7, 2025
Merged

chore: remove maintenance banner#1295
jjramirezn merged 1 commit intopeanut-walletfrom
fix/maintainance-setup

Conversation

@kushagrasarathe
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Oct 7, 2025

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

Project Deployment Preview Comments Updated (UTC)
peanut-wallet Building Building Preview Comment Oct 7, 2025 1:32pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 7, 2025

Walkthrough

Toggled underMaintenanceConfig.enableMaintenanceBanner from true to false in src/config/underMaintenance.config.ts. No other flags or logic were changed.

Changes

Cohort / File(s) Summary
Maintenance config
src/config/underMaintenance.config.ts
Flipped enableMaintenanceBanner default to false; enableFullMaintenance unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

enhancement

Suggested reviewers

  • Hugo0
  • jjramirezn

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The pull request has no author-provided description, so there is no contextual information explaining the change or its intent, leaving reviewers without any guidance; this absence of description fails to meet the requirement of being related to the changeset. Please add a brief description summarizing the change to disable the maintenance banner flag, including the file modified and the reason for removing the banner, so reviewers can understand the rationale and impact.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title “chore: remove maintenance banner” accurately and succinctly describes the primary change in the pull request, which is disabling the maintenance banner by toggling its configuration flag; it is clear, specific, and fully related to the changeset.
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 fix/maintainance-setup

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 416cfc4 and 562fb8f.

📒 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
🔇 Additional comments (1)
src/config/underMaintenance.config.ts (1)

28-28: Approve configuration change

The enableMaintenanceBanner flag is consumed in src/components/Global/Banner/index.tsx and disabling it will correctly hide the banner.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@coderabbitai coderabbitai bot added the enhancement New feature or request label Oct 7, 2025
@jjramirezn jjramirezn merged commit fea4ce8 into peanut-wallet Oct 7, 2025
6 of 8 checks passed
@vercel vercel bot temporarily deployed to Preview – peanut-wallet October 7, 2025 13:35 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants