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

Accept more then one role to check for permission #5

Merged
merged 2 commits into from
Dec 14, 2020

Conversation

vivianedias
Copy link
Contributor

One of Mapa do Acolhimento users wasnt allowed to make matches because the role permission selected to make matches was USER (2), Mapa's user had a ADMIN (1) role in the community.

Now the permissions-utils accepts an array to validate the resolver permission.

@vivianedias vivianedias self-assigned this Dec 4, 2020
@vivianedias vivianedias force-pushed the feature/accept-more-roles branch from 00fbbbc to 3c9ca94 Compare December 10, 2020 12:29
@vivianedias vivianedias reopened this Dec 11, 2020
@vivianedias vivianedias requested a review from lpirola December 11, 2020 17:37
@vivianedias vivianedias merged commit d55a87a into main Dec 14, 2020
@vivianedias vivianedias deleted the feature/accept-more-roles branch December 14, 2020 14:23
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

Successfully merging this pull request may close these issues.

2 participants