Skip to content

feat: [UIE-10166] - IAM: url params for tables#13397

Merged
aaleksee-akamai merged 5 commits intolinode:developfrom
aaleksee-akamai:UIE-10166
Feb 20, 2026
Merged

feat: [UIE-10166] - IAM: url params for tables#13397
aaleksee-akamai merged 5 commits intolinode:developfrom
aaleksee-akamai:UIE-10166

Conversation

@aaleksee-akamai
Copy link
Contributor

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

Description 📝

This PR adds the URL params to Assigned Roles and Assigned Entities tables

Changes 🔄

List any change(s) relevant to the reviewer.

  • Add params
  • Update tests
  • Cleanup

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
Screen.Recording.2026-02-13.at.12.10.44.PM.mov
Screen.Recording.2026-02-13.at.12.08.39.PM.mov

How to test 🧪

Prerequisites

(How to setup test environment)

  • use iam account or child account

Reproduction steps

(How to reproduce the issue, if applicable)

  • Go to /iam/roles/defaults/roles or /iam/roles/defaults/entity-access or /iam/users/username/details or /iam/users/username/entities
  • search/filter

Verification steps

(How to verify changes)

  • there should be url params based on your search/filter
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 marked this pull request as ready for review February 13, 2026 11:14
@aaleksee-akamai aaleksee-akamai requested a review from a team as a code owner February 13, 2026 11:14
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.

Changes look great in the UI and the behavior is solid - I tested with a bunch of combinations (search, filtering, pagination etc) and confirmed the feature enhancement. Nice one!

One quick thing (unrelated to this particular PR) that could potentially be a little confusing for a user is adding an unrelated role once a filter has been applied because the table will show no update, maybe thinking this was a no-op. I don't know if there is a good way to approach this other then resetting all filters/params when adding a new role but wanted to mention it anyway (I still think we have the right approach).

As far as the code cleanup, take it or leave it - since there is a lot of logic repeat we could:

  • consolidate search param types in the route file (it was done like that for some other components)
  • address some of the cleanups I commented on (and potentially abstract the logic in a util file)

again - nit stuff

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.

Tested the verification steps, everything works as expected. Thanks!

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Feb 18, 2026
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #5 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing865 Passing11 Skipped44m 32s

Details

Failing Tests
SpecTest
quotas-storage.spec.tsCloud Manager Cypress Tests→Quota workflow tests→Quota storage table » Quotas and quota usages display properly

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/account/quotas-storage.spec.ts"

@aaleksee-akamai aaleksee-akamai merged commit f01ef1a 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
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: Merged

Development

Successfully merging this pull request may close these issues.

4 participants

Comments