Skip to content

feat: [UIE-9270] - IAM Delegation: Account Delegations Drawer - #12970

Merged
mpolotsk-akamai merged 6 commits into
linode:developfrom
mpolotsk-akamai:UIE-9270-account-delegation-drawer
Oct 21, 2025
Merged

feat: [UIE-9270] - IAM Delegation: Account Delegations Drawer#12970
mpolotsk-akamai merged 6 commits into
linode:developfrom
mpolotsk-akamai:UIE-9270-account-delegation-drawer

Conversation

@mpolotsk-akamai

@mpolotsk-akamai mpolotsk-akamai commented Oct 8, 2025

Copy link
Copy Markdown
Contributor

Description 📝

This PR implements an Account Delegations Drawer for Account Delegation tab.

Changes 🔄

List any change(s) relevant to the reviewer.

  • Add Drawer.
  • Unit 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

Target release date 🗓️

October 21st

Preview 📷

Screenshot 2025-10-13 at 12 19 26 PM
Screenshot 2025-10-13 at 12 19 42 PM

How to test 🧪

Prerequisites

(How to setup test environment)

  • have an IAM parent/child DevCloud account (IAM Delegation (Parent/Child) flag should be enabled)
  • start MSW CRUD mode and seed some child account data

Verification steps

(How to verify changes)

  • Navigate to /iam/delegations.
  • Click the 'Update Delegations' button.
  • Verify that the drawer with delegates is visible.
  • Select additional users in the Autocomplete and click “Update.”
  • Verify that the payload is updated with the appropriate users.
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 force-pushed the UIE-9270-account-delegation-drawer branch 3 times, most recently from a68df62 to 9daaa42 Compare October 13, 2025 10:15
@mpolotsk-akamai
mpolotsk-akamai marked this pull request as ready for review October 13, 2025 10:25
@mpolotsk-akamai mpolotsk-akamai self-assigned this Oct 13, 2025
@mpolotsk-akamai mpolotsk-akamai added the IAM Delegation Implementing the new Parent/Child delegation UI label Oct 13, 2025
@mpolotsk-akamai
mpolotsk-akamai requested a review from a team as a code owner October 13, 2025 10:31

@abailly-akamai abailly-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I just did a quick code pass (haven't reviewed the UI yet) to try and improve some conventions.

Please let me know if you need any assistance in resolving some of those issues

Comment thread packages/manager/src/features/IAM/Delegations/UpdateDelegationsDrawer.tsx Outdated
Comment thread packages/api-v4/src/iam/delegation.types.ts Outdated
Comment thread packages/manager/src/features/IAM/Delegations/UpdateDelegationsDrawer.tsx Outdated
Comment thread packages/manager/src/features/IAM/Delegations/UpdateDelegationsDrawer.tsx Outdated
Comment thread packages/manager/src/mocks/presets/crud/handlers/delegation.ts Outdated
Comment thread packages/manager/src/mocks/presets/crud/handlers/delegation.ts Outdated
@mpolotsk-akamai
mpolotsk-akamai force-pushed the UIE-9270-account-delegation-drawer branch from e3717bb to 87530c5 Compare October 20, 2025 13:02
@mpolotsk-akamai
mpolotsk-akamai force-pushed the UIE-9270-account-delegation-drawer branch from 87530c5 to 54c14b1 Compare October 20, 2025 13:29
Comment thread packages/queries/src/iam/delegation.ts
@abailly-akamai
abailly-akamai self-requested a review October 21, 2025 07:58
});
queryClient.invalidateQueries({
queryKey: delegationQueries.allChildAccounts._def,
});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@mpolotsk-akamai we were missing the ALL query invalidation

@abailly-akamai abailly-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great work!

Confirmed UI, validation, query invalidation and data flow ✅

Also pushed a couple fix needed for the mock data to behave properly

@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🔺 1 failing test on test run #10 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing840 Passing11 Skipped39m 2s

Details

Failing Tests
SpecTest
clone-linode.spec.tsCloud Manager Cypress Tests→clone linode » can clone a Linode from Linode details page

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/linodes/clone-linode.spec.ts"

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Oct 21, 2025
@mpolotsk-akamai
mpolotsk-akamai merged commit ac5aa44 into linode:develop Oct 21, 2025
34 of 35 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Oct 21, 2025
corya-akamai pushed a commit to corya-akamai/manager that referenced this pull request Dec 8, 2025
…#12970)

* feat: [UIE-9270] - IAM Delegation: Account Delegation Drawer

* feat: [UIE-9270] - fix autocomplete, unit test

* cleanup

* Added changeset: IAM: Account Delegations Drawer

* feat: [UIE-9270] - IAM Delegation: review fix

* data handling fixes

---------

Co-authored-by: Alban Bailly <abailly@akamai.com>
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

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants