-
Notifications
You must be signed in to change notification settings - Fork 260
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
Use getRoleBasicInfoById when checking Administrator role #7131
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7131 +/- ##
=======================================
Coverage 32.02% 32.02%
=======================================
Files 41 41
Lines 893 893
Branches 220 216 -4
=======================================
Hits 286 286
Misses 557 557
Partials 50 50
Flags with carried forward coverage won't be shown. Click here to find out more.
|
🦋 Changeset detectedThe changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. |
PR builder started |
PR builder completed |
PR builder is not working for identity-apps PRs. |
Purpose
Use getRoleBasicInfoById instead of getRole method when checking Administrator role to avoid unnecessary DB calls.
Related Issues
wso2/product-is#21496