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

[FEAT] Filtering Manage Permissions #57

Open
mshodge opened this issue Dec 11, 2024 · 1 comment
Open

[FEAT] Filtering Manage Permissions #57

mshodge opened this issue Dec 11, 2024 · 1 comment

Comments

@mshodge
Copy link
Contributor

mshodge commented Dec 11, 2024

Hi there - apologies, won't be much help in actually implementing this as my TS knowledge is zero.

Screenshot 2024-12-11 at 14 48 45

However, some suggested improvements for the Manage Permissions app so that it only shows what is relevant to the user. That is:

Admins

  • Can see everything as is currently presented across all tabs (User Permissions, Experiment Permissions etc)

Manage Users

  • Remove ability to see User Permissions (this should just be for admins)
  • Filter experiments under Experiment Permission tab to be only those experiments they have MANAGE access on (currently everyone can see all experiments and are presented a 403 error if they click into something they don't have manage permissions for)
  • Same for Model Permissions
  • Haven't used Group Permissions yet so not sure what this should be

Edit Users and Read Users

  • Remove ability to see User Permissions (this should just be for admins)
  • Remove all experiments in Experiment Permission (unless they are a MANAGE user on one/more)
  • Remove all models in Model Permission (unless they are a MANAGE user on one/more)
  • Haven't used Group Permissions yet so not sure what this should be
@kharkevich
Copy link
Contributor

hi @mshodge, could you please check 3.2.0? how it matches to your request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants