Skip to content

Conversation

@abailly-akamai
Copy link
Contributor

@abailly-akamai abailly-akamai commented Dec 15, 2025

Description 📝

When a user want to move an entity from a role to another (Change Role drawer) but already has entities assigned to a related role, those roles are filtered out, resulting in the user being unable to switch the entity.

Changes 🔄

  • Remove already assigned roles Role filter in <ChangeRoleForEntityDrawer />

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 🗓️

12/16

How to test 🧪

Prerequisites

An IAM account with admin role

Reproduction steps

Consider assigning entities to similar roles. ex:
Screenshot 2025-12-15 at 12 23 24

  • Navigate to /iam/users//entities
  • Open the "Change Role" drawer for one of the firewalls
  • ⛔ Can't select one of the similar roles

Verification steps

  • Navigate to /iam/users//entities
  • Open the "Change Role" drawer for one of the firewalls
  • ✅ Similar roles are all available
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

@abailly-akamai abailly-akamai self-assigned this Dec 15, 2025
@abailly-akamai abailly-akamai changed the title fix: [UIE-9840] - Remove Role filter (already assigned roles) in ChangeRoleForEntityDrawer fix: [UIE-9840] - IAM: Remove Role filter (already assigned roles) in ChangeRoleForEntityDrawer Dec 15, 2025
@abailly-akamai abailly-akamai marked this pull request as ready for review December 15, 2025 11:29
@abailly-akamai abailly-akamai requested a review from a team as a code owner December 15, 2025 11:30
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing830 Passing11 Skipped36m 8s

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 Dec 15, 2025
@abailly-akamai abailly-akamai merged commit 9533b08 into linode:develop Dec 15, 2025
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Dec 15, 2025
abailly-akamai added a commit that referenced this pull request Dec 16, 2025
… `ChangeRoleForEntityDrawer` (#13201)

* don't filter out existing roles in ChangeRoleForEntityDrawer

* Added changeset: IAM: Remove Role filter (already assigned roles) in ChangeRoleForEntityDrawer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants