Skip to content

chore: disable maintenance mode post-release#2036

Merged
Hugo0 merged 1 commit into
mainfrom
chore/disable-maintenance-post-release
May 14, 2026
Merged

chore: disable maintenance mode post-release#2036
Hugo0 merged 1 commit into
mainfrom
chore/disable-maintenance-post-release

Conversation

@Hugo0

@Hugo0 Hugo0 commented May 14, 2026

Copy link
Copy Markdown
Contributor

URGENT — post-cutover maint-off

The decomplexify cutover (PR #1984 + peanut-api-ts#747) is live. Flipping enableFullMaintenance back to false.

Why this is separate: PR #1985 (maint=true) was merged into main only, never back-merged to dev. So PR #1984's dev→main merge kept main's true (3-way merge: dev didn't change the line, main did → result keeps main's value).

One-line change. Merge ASAP.

The decomplexify cutover migrations + backfill completed successfully
at 2026-05-14 19:50 UTC. BE PR #747 + FE PR #1984 merged. Flipping
maint off so users come out of the /maintenance redirect.

Why this needs a separate PR: PR #1985 (enable maint) was merged into
main but never synced to dev. When PR #1984 merged dev -> main, the
3-way merge kept main's true (dev didn't touch the line).
@vercel

vercel Bot commented May 14, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
peanut-wallet Ready Ready Preview, Comment May 14, 2026 8:09pm

Request Review

@coderabbitai

coderabbitai Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

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: 6384c9b3-4518-4baf-8ddb-01a792577a2b

📥 Commits

Reviewing files that changed from the base of the PR and between 74f5c57 and 8ef9bb5.

📒 Files selected for processing (1)
  • src/config/underMaintenance.config.ts

Walkthrough

The maintenance configuration default for full-app redirection is toggled from enabled to disabled. The enableFullMaintenance flag in underMaintenanceConfig now defaults to false, meaning the application will not automatically redirect all routes to /maintenance unless explicitly enabled at runtime.

Changes

Maintenance Feature Configuration

Layer / File(s) Summary
Disable full-app maintenance redirect by default
src/config/underMaintenance.config.ts
enableFullMaintenance default changes from true to false in the exported maintenance configuration object.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Possibly related PRs

  • peanutprotocol/peanut-ui#1985: Modifies the same enableFullMaintenance configuration flag to control full-route redirection behavior for the maintenance feature.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and accurately describes the main change: disabling maintenance mode after the release cutover is complete.
Description check ✅ Passed The description is directly related to the changeset, providing context about why the maintenance flag is being disabled post-release.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

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

@Hugo0 Hugo0 merged commit 4927532 into main May 14, 2026
15 of 18 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Code-analysis diff

Painscore total: 5648.74 → 5648.74 (0)
Findings: 0 net (+0 new, -0 resolved)

@github-actions

Copy link
Copy Markdown
Contributor

🧪 UI test report — ✅ all green

Suites

  • unit: 1073 ran, 0 failed, 0 skipped, 17.9s

📊 Coverage (unit)

metric %
statements 46.3%
branches 27.3%
functions 27.0%
lines 46.1%
⏱ 10 slowest test cases
time test
0.3s src/app/actions/__tests__/api-headers-extended.test.ts › should not include apiKey in updateUserById body
0.3s src/app/actions/__tests__/api-headers.test.ts › should include Content-Type in updateUserById
0.1s src/app/actions/__tests__/api-headers.test.ts › should include Content-Type in createOnrampForGuest
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid 9-digit US account
0.1s src/app/(mobile-ui)/qr-pay/__tests__/qr-pay-states.test.tsx › Manteca PIX form ready shows merchant card + amount input + pay button
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle too long for US account
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid ENS name
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid German IBAN
0.1s src/components/Request/__tests__/request-states.test.tsx › after link creation, QR code is no longer blurred
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid UK IBAN with spaces
📍 Inline annotations are in the **Unit test report** check above. Coverage artifact: `coverage-unit`. Generated by `.github/workflows/tests.yml`.

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