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

Allow System Admin or Team Admin to use Wrangler #91

Closed
nathanaelhoun opened this issue Aug 16, 2020 · 2 comments
Closed

Allow System Admin or Team Admin to use Wrangler #91

nathanaelhoun opened this issue Aug 16, 2020 · 2 comments

Comments

@nathanaelhoun
Copy link

For now, plugin access is allowed for either all users or users with the "Allowed Email Domain". It could be nice to allow it to System Admin or Team Admin.

For example, we can choose in the settings between "Allowed Email Domain", "System Admins", "System and Team Admins" or "All users".
For Team Admins, it may be trickier : are they authorised to move a thread from a team to another ?

@gabrieljackson
Copy link
Owner

Hey @nathanaelhoun, thanks for the suggestion. You are right that the only form of restriction is via email settings. I will look into checking for admin roles to see if that is viable or not and will get back with you.

@gabrieljackson
Copy link
Owner

Going to close this issue having partially implemented the request. The plugin now supports System Admins only, System Admins and Allowed Email Domains (which can support multiple domains or specific user emails), and All Users.

There are some edge cases supporting Team Admins with other permission settings which can lead to confusing outcomes so I am going to avoid implementing that option for now.

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