Skip to content
This repository has been archived by the owner on Feb 14, 2025. It is now read-only.

Add permission hints to responses and update UI to use them #248

Merged
merged 1 commit into from
Jan 10, 2020

Conversation

ryan-lane
Copy link
Contributor

This change adds permissions hints for the UI, so that we can change the UI behavior based on the permissions available to the end-user.

@ryan-lane ryan-lane requested a review from skiptomyliu January 10, 2020 00:57
@@ -32,6 +35,26 @@ def get_client_config():
'''
Get configuration to help clients bootstrap themselves.
'''
permissions = {
Copy link
Member

@skiptomyliu skiptomyliu Jan 10, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah nice, if I understand correctly, we may be able to leverage this later to save us some RBAC API calls by storing these default permissions once the user logs in.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@skiptomyliu yeah, these are the "global" permissions that apply to all views.

@ryan-lane ryan-lane merged commit 9bc7c9c into master Jan 10, 2020
@ryan-lane ryan-lane deleted the permissions-hints branch January 10, 2020 01:34
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants