Skip to content

fix: [UIE-10074, UIE-10076, UIE-10075] - IAM: Assigned Roles table pagination fixes#13346

Merged
abailly-akamai merged 6 commits intolinode:developfrom
abailly-akamai:UIE-10074-alt
Feb 4, 2026
Merged

fix: [UIE-10074, UIE-10076, UIE-10075] - IAM: Assigned Roles table pagination fixes#13346
abailly-akamai merged 6 commits intolinode:developfrom
abailly-akamai:UIE-10074-alt

Conversation

@abailly-akamai
Copy link
Contributor

@abailly-akamai abailly-akamai commented Jan 29, 2026

Proposed as an alternative to #13330

Description 📝

This PR fixes couple of bugs related to resetting page in Assigned Roles/Entities table when user Remove/Unassign/Change role/Assignment

Changes 🔄

List any change(s) relevant to the reviewer.

  • Update usePagination hook logic for better page resetting synchronization
  • Cleanup instances and manual pagination instances
  • Improve hook coverage

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
UIE-10074
Screen.Recording.2026-01-28.at.10.12.03.AM.mov
Screen.Recording.2026-01-28.at.10.13.25.AM.mov
UIE-10075
Screen.Recording.2026-01-28.at.10.22.19.AM.mov
Screen.Recording.2026-01-28.at.10.23.27.AM.mov
Screen.Recording.2026-01-28.at.10.24.48.AM.mov
Screen.Recording.2026-01-28.at.10.25.44.AM.mov
UIE-10076
Screen.Recording.2026-01-28.at.10.40.11.AM.mov
Screen.Recording.2026-01-28.at.10.41.03.AM.mov

How to test 🧪

Prerequisites

(How to setup test environment)

  • use iam account
  • use child account + IAM Delegation flag is enabled

Reproduction steps

(How to reproduce the issue, if applicable)

UIE-10074

  • Go to /iam/users/username/roles (or /iam/roles/defaults/roles) with 26 assigned roles
  • Go to 2. page of the table.
  • Click Remove option in the role's action menu.
  • Click Cancel button in the remove confirmation popup.

UIE-10075 - 1st issue

  • Go to /iam/users/username/roles (or /iam/roles/defaults/roles) with 26 assigned roles
  • linode_admin role is the last one on the 1st page and linode_contributer is the 1st role on the 2nd page
  • Go to 2. page of the table
  • Click Change role for linode_contributer to linode_viewer

UIE-10075 - 2nd issue

  • Go to /iam/users/username/roles (or /iam/roles/defaults/roles) with 26 assigned roles
  • Change role for any role on the 1st page

UIE-10076

  • Go to /iam/users/username/roles (or /iam/roles/defaults/roles) with 26 assigned roles
  • Go to 2. page of the table with the one entity role there
  • Click remove assignment form that role

Verification steps

(How to verify changes)

UIE-10074

  • The table should remain on the same page

UIE-10075

  • The table should remain on the same page if user changes role to a new one
  • The table should reset to the prev page if user change role to the already existing one
  • The table should remain on the same page if user changes role on the 1st page

UIE-10076

  • The table should reset to the prev page if user removes the last assignment form the 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

@abailly-akamai abailly-akamai marked this pull request as ready for review February 2, 2026 10:03
@abailly-akamai abailly-akamai requested a review from a team as a code owner February 2, 2026 10:03
Copy link
Contributor

@aaleksee-akamai aaleksee-akamai left a comment

Choose a reason for hiding this comment

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

I like your approach. Thanks for cleaning this up!

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

Cloud Manager UI test results

🔺 1 failing test on test run #6 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing864 Passing11 Skipped43m 27s

Details

Failing Tests
SpecTest
search-images.spec.tsCloud Manager Cypress Tests→Search Images » creates two images and make sure they show up in the table and are searchable

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/images/search-images.spec.ts"

@abailly-akamai abailly-akamai merged commit 03b59dc into linode:develop Feb 4, 2026
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Feb 4, 2026
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