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

Add request prominence counts in admin UI #7348

Merged
merged 2 commits into from
Oct 14, 2022

Conversation

garethrees
Copy link
Member

Make it easy to quickly see the distribution of prominence states for
requests on body and user admin pages.

The use case for this was wanting to know if a request hidden due to a
data breach was the first for the authority in question or yet another
in a long history of breaches.

We do try to minimise reduced prominence to specific messages or even
attachments after #1005 [1], so only showing request prominence counts
doesn't tell the whole story, but it's a start.

Public Body Admin Page

Screenshot 2022-10-06 at 13 15 22

User Admin Page

Screenshot 2022-10-06 at 13 14 57

Make it easy to quickly see the distribution of prominence states for
requests on body and user admin pages.

The use case for this was wanting to know if a request hidden due to a
data breach was the first for the authority in question or yet another
in a long history of breaches.

We do try to minimise reduced prominence to specific messages or even
attachments after #1005 [1], so only showing request prominence counts
doesn't tell the whole story, but it's a start.

[1] #1005
Makes more sense to render in decreasing visibility where we iterate
through the array.
@garethrees garethrees changed the title Admin prominence counts Add request prominence counts in admin UI Oct 6, 2022
@garethrees garethrees marked this pull request as ready for review October 6, 2022 13:00
@mysociety-pusher mysociety-pusher merged commit b39d7e8 into develop Oct 14, 2022
@mysociety-pusher mysociety-pusher deleted the admin-prominence-counts branch October 14, 2022 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants