Skip to content

fix: [UIE-10253], [UIE-10255], [UIE-10328], [UIE-10360] - IAM Delegate Switch Account company name fix#13433

Open
mpolotsk-akamai wants to merge 6 commits intolinode:developfrom
mpolotsk-akamai:UIE-10253-delegate-user-fix
Open

fix: [UIE-10253], [UIE-10255], [UIE-10328], [UIE-10360] - IAM Delegate Switch Account company name fix#13433
mpolotsk-akamai wants to merge 6 commits intolinode:developfrom
mpolotsk-akamai:UIE-10253-delegate-user-fix

Conversation

@mpolotsk-akamai
Copy link
Contributor

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

Description 📝

This PR updates the UI behavior for delegate users without the required account permissions and improves error handling in the Switch Account drawer.

Changes 🔄

List any change(s) relevant to the reviewer.

  • hide account name in the top menu
  • remove "Current account" from the User Menu Popover
  • update the toast message to display “Account switched.” instead of undefined
  • update capitalization for the “Switch back to your account” button
  • improve error handling in Switch Account Drawer
  • update redirect flow when switching between parent and child accounts

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

Preview 📷

Before After
Screenshot 2026-02-25 at 7 56 53 PM Screenshot 2026-02-25 at 7 50 39 PM
Screen.Recording.2026-02-25.at.7.54.31.PM.mov
Screen.Recording.2026-02-25.at.7.52.58.PM.mov

How to test 🧪

Prerequisites

(How to setup test environment)

  • use Parent Account
  • Ensure you have:
    1. A delegate user without account_viewer and account_admin roles.
    2. A Parent user without delegations (for error testing).

Reproduction steps

(How to reproduce the issue, if applicable)
Delegate user without permissions

  • Log in as a Parent user.
  • Open the User Menu.
  • Click “Switch Account.”
  • Select a delegate user without account_viewer or account_admin roles.

API error handling

  • Log in as a Parent user with no delegations.
  • Block the API request for loading child accounts (e.g., via DevTools).
  • Open the User Menu.
  • Click “Switch Account.”

Verification steps

(How to verify changes)

Delegate UI updates

  • After switching accounts, verify the toast displays “Account switched.” (not undefined).
  • Verify the account name is not displayed in the top navigation.
  • Verify “Current account” is not shown in the User Menu popover.

Error handling

  • Confirm that when the request fails, only the API error is displayed and the delegation/access error is not shown.

Redirect flow

  • Verify that switching between parent and child accounts always redirects to the Linode landing page.
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 24, 2026
@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-10253-delegate-user-fix branch from 33568b9 to 4bf06f0 Compare February 25, 2026 16:22
@mpolotsk-akamai mpolotsk-akamai added IAM (Identity & Access Management) IAM Delegation Implementing the new Parent/Child delegation UI labels Feb 25, 2026
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 6 failing tests on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
6 Failing863 Passing11 Skipped45m 43s

Details

Failing Tests
SpecTest
alerting-notification-channel-permission-tests.spec.tsCloud Manager Cypress Tests→Notification Channel Listing Page — Access Control » allows access when notificationChannels is enabled
alerting-notification-channel-permission-tests.spec.tsCloud Manager Cypress Tests→Notification Channel Listing Page — Access Control » hides the Notification Channels tab when notificationChannels is disabled
smoke-linode-landing-table.spec.tsCloud Manager Cypress Tests→linode landing checks » checks the landing page side menu items
smoke-linode-landing-table.spec.tsCloud Manager Cypress Tests→linode landing checks » checks the create menu dropdown items
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Parent to Child » can search child accounts
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/cloudpulse/alerting-notification-channel-permission-tests.spec.ts,cypress/e2e/core/linodes/smoke-linode-landing-table.spec.ts,cypress/e2e/core/parentChild/account-switching.spec.ts"

@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-10253-delegate-user-fix branch 2 times, most recently from a369f1d to b0ac008 Compare February 25, 2026 18:40
@mpolotsk-akamai mpolotsk-akamai marked this pull request as ready for review February 25, 2026 19:16
@mpolotsk-akamai mpolotsk-akamai requested a review from a team as a code owner February 25, 2026 19:16
@mpolotsk-akamai mpolotsk-akamai changed the title fix: [UIE-10253] - IAM Delegate Switch Account company name fix fix: [UIE-10253], [UIE-10255], [UIE-10328] - IAM Delegate Switch Account company name fix Feb 25, 2026
@abailly-akamai
Copy link
Contributor

@mpolotsk-akamai could we try to address the failing account-switching.spec.ts tests?

@mpolotsk-akamai mpolotsk-akamai requested a review from a team as a code owner February 26, 2026 12:08
@mpolotsk-akamai mpolotsk-akamai requested review from cliu-akamai and removed request for a team February 26, 2026 12:08
@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-10253-delegate-user-fix branch from cb36d5d to 0d541ba Compare February 26, 2026 12:22
describe('Parent/Child account switching', () => {
beforeEach(() => {
// Disable IAM delegation to use legacy child accounts flow for all tests
mockAppendFeatureFlags({ iamDelegation: false });
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jdamore-linode, quick question: for the failing E2E tests, I turned off the IAM delegation flag in the test. Does that look okay, or should we fix this another way?

Copy link
Contributor

Choose a reason for hiding this comment

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

@mpolotsk-akamai it appears the tests are still failing either way? @jdamore-linode would you mind checking? thx!

@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-10253-delegate-user-fix branch from 4176c9c to ea623b2 Compare February 27, 2026 10:17
@mpolotsk-akamai mpolotsk-akamai changed the title fix: [UIE-10253], [UIE-10255], [UIE-10328] - IAM Delegate Switch Account company name fix fix: [UIE-10253], [UIE-10255], [UIE-10328], [UIE-10360] - IAM Delegate Switch Account company name fix Feb 27, 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: Review

Development

Successfully merging this pull request may close these issues.

4 participants