Skip to content

Remove maintenance mode#1031

Merged
skyfallwastaken merged 1 commit intomainfrom
maintenance-bye
Mar 2, 2026
Merged

Remove maintenance mode#1031
skyfallwastaken merged 1 commit intomainfrom
maintenance-bye

Conversation

@skyfallwastaken
Copy link
Member

Removes the maintenance mode middleware, controller, route, error controller integration, and static HTML page.

Copilot AI review requested due to automatic review settings March 2, 2026 20:05
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Copy link
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

Removes the Rails “maintenance mode” feature end-to-end (middleware, controller, route, error-controller integration) along with the static maintenance HTML page.

Changes:

  • Deleted the static public/maintenance.html page and the supporting MaintenanceController.
  • Removed MaintenanceModeMiddleware and its insertion into the Rails middleware stack.
  • Removed the /maintenance route and the ErrorsController hook that rendered the maintenance page.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
public/maintenance.html Removes the static maintenance landing page.
lib/maintenance_mode_middleware.rb Deletes the middleware that returned a 503 maintenance page based on MAINTENANCE_MODE.
config/routes.rb Removes the /maintenance route.
config/application.rb Removes requiring/inserting the maintenance middleware into the stack.
app/controllers/maintenance_controller.rb Removes the controller that served the maintenance page.
app/controllers/errors_controller.rb Removes the maintenance-page rendering before-action.

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

@skyfallwastaken skyfallwastaken merged commit 9c91d71 into main Mar 2, 2026
18 checks passed
@skyfallwastaken skyfallwastaken deleted the maintenance-bye branch March 2, 2026 20:07
@skyfallwastaken
Copy link
Member Author

i am pretty sure this is not even the correct fix but this is to just rule out the repo acting up in case if it's coolify

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.

2 participants