Skip to content

Comments

feat: [UIE-10184] - IAM: add permissions to UI#13409

Merged
aaleksee-akamai merged 4 commits intolinode:developfrom
aaleksee-akamai:UIE-10184
Feb 20, 2026
Merged

feat: [UIE-10184] - IAM: add permissions to UI#13409
aaleksee-akamai merged 4 commits intolinode:developfrom
aaleksee-akamai:UIE-10184

Conversation

@aaleksee-akamai
Copy link
Contributor

@aaleksee-akamai aaleksee-akamai commented Feb 17, 2026

Description 📝

This PR disables/enables UI for default roles and account delegations based on permissions.

Note: Delegate users also will be able to add/remove the delegate roles

Changes 🔄

List any change(s) relevant to the reviewer.

  • Add permissions
  • 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 parent/child/delegate account

Verification steps

(How to verify changes)

  • child/delegate account_admin and account_viewer should be able to view default roles and entities.
  • child/delegate account_admin should be able to update default roles and entities.
  • parent account_admin and account_viewer should be able to view account delegations.
  • parent account_admin should be able to update account delegations.
  • unauthorized users should see a notification banner in Account Delegations.
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 self-assigned this Feb 17, 2026
@aaleksee-akamai aaleksee-akamai added the IAM Delegation Implementing the new Parent/Child delegation UI label Feb 17, 2026

const userName =
(isProxyOrDelegateUserType ? parentProfile : profile)?.username ?? '';
const userName = (isProxyUserType ? parentProfile : profile)?.username ?? '';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we have a ticket for that UIE-10223, I've done it here in order to see that you've logged as a delegate user for testing purpose

However, I don't know why it was done for proxy users

@aaleksee-akamai aaleksee-akamai marked this pull request as ready for review February 18, 2026 09:23
@aaleksee-akamai aaleksee-akamai requested a review from a team as a code owner February 18, 2026 09:23
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #4 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing867 Passing11 Skipped35m 23s

Details

Failing Tests
SpecTest
account-switching.spec.tsCloud Manager Cypress Tests→Parent/Child account switching→From Parent to Child » can search child accounts

Troubleshooting

Use this command to re-run the failing tests:

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

Copy link
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.

Looks good to me. Tested parent/child with and without permissions. Everything works as expected. Thanks!

Copy link
Contributor

@abailly-akamai abailly-akamai left a comment

Choose a reason for hiding this comment

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

This looks great - thx @aaleksee-akamai

Although it's unlikely for this PR, due to the the recent regressions we encountered with the legacy experience we should triple down testing all cases.

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Feb 19, 2026
@aaleksee-akamai
Copy link
Contributor Author

This looks great - thx @aaleksee-akamai

Although it's unlikely for this PR, due to the the recent regressions we encountered with the legacy experience we should triple down testing all cases.

@abailly-akamai , I think we need to merge your PR into develop (after the release) and check if it helps

@aaleksee-akamai aaleksee-akamai merged commit c68e4c6 into linode:develop Feb 20, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Feb 20, 2026
dchyrva-akamai pushed a commit to dchyrva-akamai/manager that referenced this pull request Feb 23, 2026
* feat: [UIE-10184] - IAM: add permissions to UI

* typo

* Added changeset: IAM Parent/Child: add permissions to UI
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

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

4 participants