Skip to content

UISAUTHCOM-96 (Sunflower BF) Some capabilities not shown when editing a role but shown in its detail view - #153

Draft
JohnC-80 wants to merge 5 commits into
b2.0from
UISAUTHCOM-96v3
Draft

UISAUTHCOM-96 (Sunflower BF) Some capabilities not shown when editing a role but shown in its detail view#153
JohnC-80 wants to merge 5 commits into
b2.0from
UISAUTHCOM-96v3

Conversation

@JohnC-80

@JohnC-80 JohnC-80 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

UISAUTHCOM-96

Purpose

The Role Edit page is missing application entries that are present on the Role Details page.

Approach

Added another call of useRoleCapabilites with expand=true to fill out capabilitiesAppIds from the associated, inherited capabilities, not exclusively the directly selected capabilities.

If https://folio-org.atlassian.net/browse/MODROLESKC-408 is backported, this will be unnecessary, since the directly assigned capabilities can be discerned from the inherited ones in a single call with expand=true.

Check/uncheck behavior of Capability Sets is maintained and persists.

Adding isFetching to output from hooks so that we can wait for initial data to be loaded.

The capabilities endpoint in >= Trillium includes the direct field on capabilities, so it's must easier for the UI to distinguish directly-assigned capabilities vs inherited ones.

Tests updated to reflect the current state of the code.

Bugfix: (Same apps showing up in tables on Edit screen as appear on the Details view.)

2026-08-13_09h22_47.mp4

Capability Sets selection/deselection:

2026-08-13_09h59_34.mp4

Re-edit

2026-08-13_10h02_19.mp4

Inherited/direct-assignment Capabilities

2026-08-13_10h06_00.mp4

@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown

Jest Unit Test Results

  1 files  ±0   51 suites  ±0   1m 35s ⏱️ -7s
167 tests +6  166 ✅ +6  1 💤 ±0  0 ❌ ±0 
175 runs  +6  174 ✅ +6  1 💤 ±0  0 ❌ ±0 

Results for commit f9b918f. ± Comparison against base commit 2412d70.

♻️ This comment has been updated with latest results.

@sonarqubecloud

Copy link
Copy Markdown

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant