Admin dashboard redesign #264
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This moves app config to a "Settings" page and the application monitor to a "Monitor" page. It also reworks the admin navigation bar a bit and adds some instance stats on the dashboard. Ref T694
This adds a new config option that signifies the admin doesn't need to see deeply technical things, like application resource usage. In the [app] section, set forest = true to enable this. Ref T694
- Tweak the copy - Include link to download latest release - Change the version status design a bit - Restyle some text
When a WriteFreely update is available, indicate this on the "Updates" navigation item Ref T572
This keeps track when automated update check fails, and displays a relevant message to the admin on /admin/updates Ref T572
Logs requests when --debug enabled, and always logs errors from the check. Ref T572
Previously, we'd wait for a response before finishing app initialization, meaning an overall slower startup. This fixes that. Ref T572
- Add a description to each config item - Change item names to make more sense Ref T694
|
I think this is a pretty good stopping place for this pull request. Would love to hear any feedback on these changes! |
|
This looks great! |
|
Thanks @dariusk! Really glad you like it. @cjeller1592 some documentation to add, so we can link from the admin Settings page:
|
|
@cjeller1592 I moved those documentation topics to T728. Will get this merged and then we can update the app in another PR once all of that is live. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reworking the admin dashboard to be more organized, straightforward, and to optionally hide unnecessary information for non-technical admins in hosted environments like Write.as for Teams.
Less-technical dashboard
This will close T694.
This hides the app Monitor tab on the dashboard if instance admin sets the following config value:
forest = true(note: experimental).This makes the App Configuration page more user-friendly by changing the user-facing language and adding descriptions to each item.
New Admin Settings page
New Dashboard
New Users page
Update page
Enabled with:
update_checks = true(note: experimental).Automated "new version" checks
It also contains improvements to the automated update check system, both user-facing and on the backend (work started on #175 and T572):
/admin/updatespage