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

Admin dashboard redesign #264

Open
wants to merge 18 commits into
base: develop
from
Open

Admin dashboard redesign #264

wants to merge 18 commits into from

Conversation

@thebaer
Copy link
Member

thebaer commented Feb 14, 2020

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

screencapture-localhost-7879-admin-settings-2020-02-25-14_26_58

New Dashboard

screencapture-localhost-7879-admin-2020-02-25-14_27_21

New Users page

Screenshot from 2020-02-25 14-43-31

Update page

screencapture-localhost-7879-admin-updates-2020-02-25-14_42_49

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):

  • Indicates a new update is available in the admin navigation bar
  • Shows "last check" time in local timezone, like blog posts do
  • Mentions update check error to admin when one occurs
  • Mentions that update checks are disabled, when true and an admin navigates to the /admin/updates page
  • Adds logging around update checks
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
@thebaer thebaer added the design label Feb 14, 2020
@thebaer thebaer added this to the 0.12 milestone Feb 14, 2020
thebaer added 2 commits Feb 14, 2020
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
@thebaer thebaer changed the title Admin Dashboard Redesign Admin dashboard redesign Feb 14, 2020
thebaer added 12 commits Feb 14, 2020
- 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
@thebaer

This comment has been minimized.

Copy link
Member Author

thebaer commented Feb 25, 2020

I think this is a pretty good stopping place for this pull request. Would love to hear any feedback on these changes!

@dariusk

This comment has been minimized.

Copy link
Contributor

dariusk commented Feb 25, 2020

This looks great!

@thebaer

This comment has been minimized.

Copy link
Member Author

thebaer commented Feb 26, 2020

Thanks @dariusk! Really glad you like it.

@cjeller1592 some documentation to add, so we can link from the admin Settings page:

  • Choosing a name for your site
  • Choosing a "Landing Page" value
  • Choosing a "max blogs per user" value (unlimited, only one, or limited)
  • Controlling registration (open, closed, invite-only for admins, invite-only for users)
  • What it means to run a private instance vs. public instance
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.