Skip to content

fix(admin-ui): resolve access management inconsistencies across modules (#2659)#2664

Closed
faisalsiddique4400 wants to merge 11 commits intomainfrom
admin-ui-issue-2659
Closed

fix(admin-ui): resolve access management inconsistencies across modules (#2659)#2664
faisalsiddique4400 wants to merge 11 commits intomainfrom
admin-ui-issue-2659

Conversation

@faisalsiddique4400
Copy link
Contributor

fix(admin-ui): resolve access management inconsistencies across modules (#2659)

Summary

Access management in the Admin UI was not functioning consistently. Permission-based UI controls were incorrectly rendered, resulting in missing action buttons and restricted edit capabilities across multiple modules.

Issues Observed

  • Action buttons were missing on search pages such as Clients and Scopes, even when appropriate permissions were assigned.
  • Certain pages such as Settings and Cedarling were not editable despite valid access rights.
  • Permission checks were not consistently synchronized with component rendering.

Root Cause

  • Inconsistent permission evaluation logic across components.
  • Conditional rendering not fully aligned with resolved access states.
  • UI not re-evaluating permissions correctly after state updates.

Fix Details

  • Standardized permission validation logic across affected modules.
  • Corrected conditional rendering of action buttons based on access rights.
  • Ensured edit permissions are properly evaluated on Settings and Cedarling pages.
  • Improved synchronization between access control state and UI rendering lifecycle.

Result

  • Buttons now appear correctly based on assigned permissions.
  • Editable pages reflect accurate access rights.
  • Access management behaves consistently across the Admin UI.

Additional Notes

This fix improves overall RBAC stability and ensures reliable enforcement of permission-based UI behavior.

🔗 Ticket

Closes: #2659

@faisalsiddique4400 faisalsiddique4400 marked this pull request as draft February 19, 2026 13:42
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch admin-ui-issue-2659

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mo-auto mo-auto added comp-admin-ui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality labels Feb 19, 2026
@sonarqubecloud
Copy link

@faisalsiddique4400 faisalsiddique4400 deleted the admin-ui-issue-2659 branch February 24, 2026 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp-admin-ui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(admin-ui): access management is not working in admin-ui

2 participants