Skip to content

Comments

fix: [UIE-10203], [UIE-10223], [UIE-10237] - IAM Delegate Users bug fix#13399

Merged
mpolotsk-akamai merged 2 commits intolinode:developfrom
mpolotsk-akamai:UIE-10237-users-delegations-table-fix
Feb 23, 2026
Merged

fix: [UIE-10203], [UIE-10223], [UIE-10237] - IAM Delegate Users bug fix#13399
mpolotsk-akamai merged 2 commits intolinode:developfrom
mpolotsk-akamai:UIE-10237-users-delegations-table-fix

Conversation

@mpolotsk-akamai
Copy link
Contributor

@mpolotsk-akamai mpolotsk-akamai commented Feb 13, 2026

Description 📝

This PR fixes issues related to account switching and delegations: it ensures the default landing page is the Linodes page, correctly displays the delegate username in the top menu, fixes pagination bug in the User Delegations table.

Changes 🔄

List any change(s) relevant to the reviewer.

  • Update the default landing page after switching accounts.
  • Display the delegate user’s username instead of the parent user’s username in the top menu.
  • Fix a pagination bug where pagination disappears after changing the page size to 50.

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 Parent account

Verification steps

(How to reproduce the issue, if applicable)

  • go to any page(ex. /firewalls)
  • open User menu
  • click Switch Account and select child account from the list
  • verify that after switching account it redirects to the Linode landing page
  • verify that delegate user’s username in displayed in top menu

Pagination:

  • as parent user go to /iam/users/[username]/delegations for a user with more than 25 delegations.
  • change page size to 50.
  • verify that Pagination is visible.
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 Feb 13, 2026
@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-10237-users-delegations-table-fix branch from 3f983a1 to db3b36b Compare February 13, 2026 15:08
@mpolotsk-akamai mpolotsk-akamai added IAM (Identity & Access Management) IAM Delegation Implementing the new Parent/Child delegation UI labels Feb 13, 2026
@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-10237-users-delegations-table-fix branch 2 times, most recently from a5676ee to aa25a0f Compare February 18, 2026 11:22
@mpolotsk-akamai mpolotsk-akamai marked this pull request as ready for review February 18, 2026 11:33
@mpolotsk-akamai mpolotsk-akamai requested a review from a team as a code owner February 18, 2026 11:33
});
onClose(event);
location.reload();
location.replace('/linodes');
Copy link
Contributor

Choose a reason for hiding this comment

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

How was the decision made to redirect to /linodes ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was agreed upon in the Figma discussion (UIE-10203), where /linodes was chosen as the default landing page.

Copy link
Contributor

Choose a reason for hiding this comment

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

Just heads up that the default page for managed users isn't /linodes tho i am not sure it is super important

</TableRow>
))}
{(childAccounts?.results ?? 0) > pagination.pageSize && (
{(childAccounts?.results ?? 0) > MIN_PAGE_SIZE && (
Copy link
Contributor

Choose a reason for hiding this comment

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

just curious why it's not a table + footer ( like in AccountDelegations or Users)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This wasn’t part of the changes in this PR, so I don’t have context on the original decision. We can handle the refactoring in a separate ticket if needed.

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Feb 20, 2026
@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-10237-users-delegations-table-fix branch from 888d660 to 37fb641 Compare February 20, 2026 14:04
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #7 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing867 Passing11 Skipped37m 47s

Details

Failing Tests
SpecTest
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Child to Child » can switch to another Child account as a Proxy user

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/parentChild/account-switching.spec.ts"

@mpolotsk-akamai mpolotsk-akamai merged commit 039f29c into linode:develop Feb 23, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Feb 23, 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.

4 participants