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

[#1636] Show AlaveteliConfiguration in admin interface #8280

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

garethrees
Copy link
Member

Relevant issue(s)

Fixes #1636

What does this do?

Extract AdminGeneralController#debug to Admin::DebugController#index, tidy up its view, and render a sanitised version ofAlaveteliConfiguration so that admins can view the configuration without a server ssh session.

Why was this needed?

Helps less technical users and admins of Alavetelis hosted by mySociety.

Playing around with speaking at chatGPT after work because I was annoyed at having to ssh to figure out the AlaveteliConfiguration.reply_very_late_after_days value.

Implementation notes

Weird issue with the routing – will try to resolve that before merge

Screenshots

Screenshot 2024-05-29 at 18 48 22

config/routes.rb Outdated Show resolved Hide resolved
@garethrees garethrees force-pushed the 1636-show-config-in-admin branch 3 times, most recently from 43f5a85 to 59518c9 Compare June 14, 2024 16:15
@garethrees garethrees marked this pull request as ready for review June 14, 2024 16:15
@garethrees garethrees force-pushed the 1636-show-config-in-admin branch from 59518c9 to 16fe5b1 Compare June 14, 2024 17:04
@garethrees garethrees force-pushed the 1636-show-config-in-admin branch from 16fe5b1 to 5cf52cf Compare July 18, 2024 19:47
Extract non-resourceful action to its own controller.
* Clean up the source code formatting
* Add bootstrap table classes
* Add a table-debug class for nicer font formatting
Makes it easier for less technical users to look up specific
configuration values – or indeed, save technical users opening an ssh
session.

Fixes #1636
@garethrees garethrees force-pushed the 1636-show-config-in-admin branch from 5cf52cf to c36b65e Compare July 18, 2024 19:50
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.

Add page to admin interface for displaying the configured values in the config file
2 participants