Skip to content

adding maintenance mode check at start of domain middleware#2369

Merged
forgetso merged 2 commits intomainfrom
fix/maintenance-mode-domain-middleware
May 6, 2026
Merged

adding maintenance mode check at start of domain middleware#2369
forgetso merged 2 commits intomainfrom
fix/maintenance-mode-domain-middleware

Conversation

@HughParry
Copy link
Copy Markdown
Contributor

No description provided.

@HughParry HughParry requested a review from forgetso as a code owner January 28, 2026 14:55
Copilot AI review requested due to automatic review settings January 28, 2026 14:55
@HughParry HughParry requested a review from goastler as a code owner January 28, 2026 14:55
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a maintenance mode check at the beginning of the domain middleware to skip domain validation when the system is in maintenance mode.

Changes:

  • Added early return in domain middleware when maintenance mode is active
  • Imported getMaintenanceMode function to check maintenance status
  • Added changeset documenting the patch

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
packages/provider/src/api/domainMiddleware.ts Added maintenance mode check with early return to skip domain validation
.changeset/silver-tires-fold.md Added changeset entry documenting the maintenance mode feature

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/provider/src/api/domainMiddleware.ts
@forgetso forgetso merged commit 1b35548 into main May 6, 2026
9 checks passed
@forgetso forgetso deleted the fix/maintenance-mode-domain-middleware branch May 6, 2026 11:47
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.

3 participants