Skip to content

feat: [UIE-10182, UIE-10168] - IAM Delegations: notifications and error state#13374

Merged
aaleksee-akamai merged 3 commits intolinode:developfrom
aaleksee-akamai:UIE-10182
Feb 9, 2026
Merged

feat: [UIE-10182, UIE-10168] - IAM Delegations: notifications and error state#13374
aaleksee-akamai merged 3 commits intolinode:developfrom
aaleksee-akamai:UIE-10182

Conversation

@aaleksee-akamai
Copy link
Copy Markdown
Contributor

Description 📝

This PR adds an error state for default roles tables and the notification banners when change role/update entities.

Changes 🔄

List any change(s) relevant to the reviewer.

  • Add an error state
  • Add tests
  • Add notification banners

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

Reproduction steps

(How to reproduce the issue, if applicable)

  • go to /iam/roles/defaults/roles and to /iam/roles/defaults/entity-access

  • block /iam/delegation/default-role-permissions endpoint

  • go to /iam/roles/defaults/roles or /iam/users/username/roles

  • change role or update entities for a role

Verification steps

(How to verify changes)

  • There should be an error state if api fails
  • There is a notification after completing an action for role
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

@aaleksee-akamai aaleksee-akamai requested a review from a team as a code owner February 6, 2026 09:45
@aaleksee-akamai aaleksee-akamai requested review from bnussman-akamai and removed request for a team February 6, 2026 09:46
@linode-gh-bot
Copy link
Copy Markdown

Cloud Manager UI test results

🎉 866 passing tests on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing866 Passing11 Skipped37m 37s

Copy link
Copy Markdown
Contributor

@mpolotsk-akamai mpolotsk-akamai left a comment

Choose a reason for hiding this comment

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

Tested the verification steps ✅
Works as expected.

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Feb 6, 2026
@aaleksee-akamai aaleksee-akamai merged commit 242c740 into linode:develop Feb 9, 2026
35 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Feb 9, 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

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants