Skip to content

upcoming: [UIE-9461] IAM Parent/Child - server side filter on Switch Account Drawer#13318

Merged
mpolotsk-akamai merged 8 commits intolinode:developfrom
mpolotsk-akamai:UIE-9460-replace-query
Feb 13, 2026
Merged

upcoming: [UIE-9461] IAM Parent/Child - server side filter on Switch Account Drawer#13318
mpolotsk-akamai merged 8 commits intolinode:developfrom
mpolotsk-akamai:UIE-9460-replace-query

Conversation

@mpolotsk-akamai
Copy link
Contributor

@mpolotsk-akamai mpolotsk-akamai commented Jan 26, 2026

Description 📝

This PR added server side filtering for Switch Account drawer

Changes 🔄

List any change(s) relevant to the reviewer.

  • Update filtering to use X-Filters and params.
  • Added table with pagination to the switch account drawer for IAM Delegation.
  • Updated akamai-cds-react-components.
  • Fixed search-related issues in the drawer.
  • Fixed incorrect search state after reopening the drawer.
  • Update tests.

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

How to test 🧪

Prerequisites

(How to setup test environment)

  • Use IAM Parent account

Verification steps

(How to verify changes)

  • navigate to a Switch Account Drawer and click "Switch Account" button
  • confirm X-Filters is working properly
  • table is visible
  • pagination is working correctly
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@mpolotsk-akamai mpolotsk-akamai self-assigned this Jan 26, 2026
@mpolotsk-akamai mpolotsk-akamai added IAM (Identity & Access Management) IAM Delegation Implementing the new Parent/Child delegation UI labels Jan 26, 2026
@mpolotsk-akamai mpolotsk-akamai marked this pull request as ready for review January 26, 2026 14:01
@mpolotsk-akamai mpolotsk-akamai requested a review from a team as a code owner January 26, 2026 14:01
@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-9460-replace-query branch 3 times, most recently from 5af6080 to 97b69c1 Compare February 5, 2026 13:52
? errors.allChildAccountsError
: errors.childAccountInfiniteError;

if (hasError) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved this to the SwitchAccountDrawer component since it’s needed for both the List and Table.

@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-9460-replace-query branch 2 times, most recently from d1e9d05 to aa9e511 Compare February 10, 2026 13:34
onPageSizeChange={(
e: CustomEvent<{ page: number; pageSize: number }>
) => handlePageSizeChange(Number(e.detail.pageSize))}
page={page}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bnussman-akamai , I have a question about pagination from usePaginationV2:

we use a table inside the drawer, and with usePaginationV2 we need to provide a currentRoute. That’s not straightforward in this case because it’s a user menu. I tried using location and useMatch, but it didn’t help. Is there a way to use it here?

I’d appreciate any suggestions!

Copy link
Member

@bnussman-akamai bnussman-akamai Feb 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm. I'm wondering if currentRoute is even necessary. We could make currentRoute optional in usePaginationV2 and that would just cause usePaginationV2 to stay on the same route.

But if we did that, we then would have to deal with query params lingering after the Switch Account drawer is closed and weird stuff like that.

In this case, using state for page and pageSize like this PR does might be the best solution because using query params for a drawer might not be desired in the first place

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 866 passing tests on test run #10 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing866 Passing11 Skipped42m 28s

Copy link
Contributor

@aaleksee-akamai aaleksee-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks!

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Feb 13, 2026
@mpolotsk-akamai mpolotsk-akamai merged commit 32c4c5e into linode:develop Feb 13, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

IAM Delegation Implementing the new Parent/Child delegation UI IAM (Identity & Access Management)

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

5 participants