-
Notifications
You must be signed in to change notification settings - Fork 36
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
permissions selection with multiselect #94
permissions selection with multiselect #94
Conversation
Awesome! Thanks for the PR, I'll review it as soon as possible |
Hi @scicco , I tested the PR and saw some issues:
Would you mind updating the PR to use this package instead? |
I would like to add a comment here: many providers use simple names to fill the matrix. None => () In this case a simple combo is enough |
@mastrogiovanni what if I want to switch to the "advanced mode"? |
I image a simple flag on top, when checked is "advanced" mode. In reality the single combo solution is also a nice solution to switch permissions directly in the user list table. |
Hello, sorry for the delay in the response, I'm having issues switching from the former component to the one you mentioned, mainly because I'm not well versed with vue and typescript in general: the new select expects having options with label and value while we simply use If I manage to resolve this, I'll also work on the other issues you mentioned |
0726865
to
1d2df85
Compare
5b4a478
to
f5e3cc0
Compare
f5e3cc0
to
9f1a097
Compare
Kudos @scicco for the contribution and kudos @zAlweNy26 for the maintaining Proud of you |
Description
This commit will change permission management by using the vue-select library instead of checkboxes and address #95
Type of change
Please delete options that are not relevant.
Checklist: