-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: #1145 use cached access for applications list #1209
feat: #1145 use cached access for applications list #1209
Conversation
ianliuwk1019
commented
Feb 21, 2024
•
edited
Loading
edited
- Only do "refreshCachedUserAccess" to update access when user login.
- Refactor on AuthService.ts to migrate user related content to FamLoginUserState.ts
- Quick addition on showing user access levels on ProfileSideBar.
- Read "applicationsUserAdministers" from cached FamLoginUserState for display on ManagePermission page.
- Read "applicationOptions" from cached FamLoginUserState for display on GrantApplicationAdmin page.
- Remove "fetchApplications" from route and from "fetchData", no need anymore.
… on grant admin page.
Thanks Ian! Looks great! Just some small comment for simplify and clarification. Also maybe Nick @NickSaglioni if you want to check on this pr, Ian is getting the application options inside the component now instead of preparing it in the router handler, just regarding of your question this morning. Thanks so much! |
Quality Gate passed for 'nr-forests-access-management_admin'Issues Measures |