Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update maintenance page #2526

Merged
merged 17 commits into from
Dec 17, 2024
Merged

Update maintenance page #2526

merged 17 commits into from
Dec 17, 2024

Conversation

vertism
Copy link
Contributor

@vertism vertism commented Dec 16, 2024

Description

Updates text on maintenance page, and changes the enabling mechanism to be an ENV var (MAINTENANCE_MODE).

Adds alert banner with details of upcoming maintenance.

RSpec.describe ApplicationController, type: :controller do
let(:user) { create(:user) }

xdescribe "#maintenance_mode" do
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Disabled these new tests as they appear to be causing test failures elsewhere. Will investigate this at a later time.

Copy link
Contributor

@lucas-shaw lucas-shaw left a comment

Choose a reason for hiding this comment

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

LGTM

@vertism vertism merged commit 13afdd9 into main Dec 17, 2024
14 checks passed
@vertism vertism deleted the update-maintenance-page branch December 17, 2024 09:16
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