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.

Sign up for GitHub

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

Jump to bottom

feat(security): get user permissions API #1620

Merged
merged 7 commits into from
Apr 23, 2021
Merged

feat(security): get user permissions API #1620

merged 7 commits into from
Apr 23, 2021

Conversation

dpgaspar
Copy link
Owner

@dpgaspar dpgaspar commented Apr 23, 2021 •
edited
Loading

Description

Add a new security manager API method to fetch a user permissions, these permissions can be a mix between builtin roles and DB roles. The actual permission fetching from the DB is efficient (single query), avoids the N+1 issue.

get_user_permissions

ADDITIONAL INFORMATION

  • Has associated issue:
  • Is CRUD MVC related.
  • Is Auth, RBAC security related.
  • Changes the security db schema.
  • Introduces new feature
  • Removes existing feature

@dpgaspar dpgaspar changed the title Feat/sec get user permissions feat(security): get user permissions API Apr 23, 2021
@dpgaspar dpgaspar merged commit 159de35 into master Apr 23, 2021
@dpgaspar dpgaspar deleted the feat/sec-get-user-permissions branch April 23, 2021 17:59
@dpgaspar dpgaspar mentioned this pull request Apr 26, 2021
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dpgaspar