-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
Hide/show UI elements based on permissions #120
Comments
Particular places in code recommended by mr0grog that would be related to this issue: |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in seven days if no further activity occurs. If it should not be closed, please comment! Thank you for your contributions. |
Still a thing that has to happen. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in seven days if no further activity occurs. If it should not be closed, please comment! Thank you for your contributions. |
Added |
This depends on actually having permissions implemented in the API (edgi-govdata-archiving/web-monitoring-db#34), so we can’t quite do this work yet.
Once the API returns permission information for a user, we should use that to limit/modify what we display:
A user might have all their basic view permissions revoked, in which case we should not request any data at all and display a message.
A user may not have permission to view annotations, in which case we should hide the annotation form/view (kind of like we do now when not logged in, but we should change the message)
A user may have permission to view annotations, but not make new ones, so we should have a view-only/disabled version of the annotation form.
The text was updated successfully, but these errors were encountered: