Skip to content

docs(FR-2909): align RBAC manual with revoke confirmation modal and refresh screenshots#7463

Open
ironAiken2 wants to merge 1 commit into
05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_namesfrom
05-18-docs_fr-2909_align_rbac_manual_with_revoke_confirmation_modal_and_refresh_screenshots
Open

docs(FR-2909): align RBAC manual with revoke confirmation modal and refresh screenshots#7463
ironAiken2 wants to merge 1 commit into
05-15-docs_rbac_align_with_revoke_remove_confirmation_flow_and_localized_scope_namesfrom
05-18-docs_fr-2909_align_rbac_manual_with_revoke_confirmation_modal_and_refresh_screenshots

Conversation

@ironAiken2
Copy link
Copy Markdown
Contributor

@ironAiken2 ironAiken2 commented May 18, 2026

Resolves #7454(FR-2909)

Follow-up to #7446. Reconciles the RBAC management user manual (en/ja/ko/th) with what FR-2819 / FR-2825 actually ship.

Changes

  • Version bump: required Backend.AI Manager version 25.4.026.4.0 in the RBAC note (rbac_management.md + admin_menu.md, all 4 languages)
  • Structure cleanup: removed the "Role Status Management" subsection that was misplaced under "View Role Details" — its content describes role-list row actions, and Active/Inactive is already covered by the Role List status filter
  • Revoke flow rewrite: the "Revoke Users from a Role" UI changed from a lightweight popconfirm to an irreversible typed-confirmation modal (BAIConfirmModalWithInput). The section now documents the per-language confirmation string (en: Permanently Delete / ko: 영구 삭제 / ja: 完全に削除 / th: ลบถาวร) and adds a :::danger irreversibility note
  • Screenshots recaptured from the live UI (replacing stale / hand-built images), all 4 languages:
    • rbac_role_detail_drawer.png, rbac_role_scope_tab.png — new localized tab labels (Scopes / Permissions / Role Assignments)
    • rbac_revoke_popconfirm.png — new typed-confirmation modal
    • rbac_permission_modal.png — real Add Permission modal (captured via project-2de2b969-member)
  • All 4 language variants kept structurally identical

Checklist: (if applicable)

  • Documentation
  • Minium required manager version
  • Specific setting for review (eg., KB link, endpoint or how to setup)
  • Minimum requirements to check during review
  • Test case(s) to demonstrate the difference of before/after

🤖 Generated with Claude Code

…efresh screenshots

Resolves #7454(FR-2909)

Follow-up to #7446. Reconciles the RBAC management manual (en/ja/ko/th) with what FR-2819 / FR-2825 actually ship.

- Bump required Backend.AI Manager version 25.4.0 -> 26.4.0 in the RBAC note (rbac_management.md + admin_menu.md, all 4 languages)
- Remove the "Role Status Management" subsection that was misplaced under "View Role Details" (its content describes role-list row actions; Active/Inactive is already covered by the Role List status filter)
- Rewrite "Revoke Users from a Role": the UI changed from a lightweight popconfirm to an irreversible typed-confirmation modal (BAIConfirmModalWithInput). Documents the per-language confirmation string (en: Permanently Delete / ko: 영구 삭제 / ja: 完全に削除 / th: ลบถาวร) and adds a :::danger irreversibility note
- Recapture screenshots from the live UI, replacing stale / hand-built images: rbac_role_detail_drawer.png and rbac_role_scope_tab.png (new localized tab labels Scopes/Permissions/Role Assignments), rbac_revoke_popconfirm.png (new typed modal), rbac_permission_modal.png (real Add Permission modal captured via project-2de2b969-member)
- All 4 language variants kept structurally identical

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor Author

ironAiken2 commented May 18, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • flow:merge-queue - adds this PR to the back of the merge queue
  • flow:hotfix - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 100~500 LoC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant