Skip to content

[DRAFT] feat: [UIE-9051] - IAM RBAC block non-beta route access - #12643

Closed
corya-akamai wants to merge 5 commits into
linode:developfrom
corya-akamai:UIE-9051-block-non-beta-iam
Closed

corya-akamai wants to merge 5 commits into
linode:developfrom
corya-akamai:UIE-9051-block-non-beta-iam

Conversation

@corya-akamai

Copy link
Copy Markdown
Contributor

Description 📝

Block non-beta users from the IAM screens and redirect to the account screens accordingly. Redirect beta users from the account screens to the IAM screens respectively.

Changes 🔄

  • Block non-beta users from the IAM screens and redirect to the account screens accordingly.
  • Redirect beta users from the account screens to the IAM screens respectively.

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

Preview 📷

How to test 🧪

Prerequisites

  • Devcloud IAM account
  • Devcloud non-IAM account

Reproduction steps

Navigate to routes

  • /iam/users <-> /account/users
  • /iam/roles <-> (no equivalent) /account/users
  • /iam/users/corya-iam/details <-> /account/users/corya-iam/profile
  • /iam/users/corya-iam/roles <-> /account/users/corya-iam/permissions
  • /iam/users/corya-iam/entities <-> /account/users/corya-iam/permissions

Verification steps

Verify redirect

  • /iam/users <-> /account/users
  • /iam/roles <-> (no equivalent) /account/users
  • /iam/users/corya-iam/details <-> /account/users/corya-iam/profile
  • /iam/users/corya-iam/roles <-> /account/users/corya-iam/permissions
  • /iam/users/corya-iam/entities <-> /account/users/corya-iam/permissions
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

Copy link
Copy Markdown
Contributor

@corya-akamai see #12644 for an alternative handling of redirects at the route definition level, which may be what we want

@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🔺 3 failing tests on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
3 Failing702 Passing4 Skipped131m 10s

Details

Failing Tests
SpecTest
qemu-reboot-upgrade-notice.spec.tsCloud Manager Cypress Tests→QEMU reboot upgrade notification » should display maintenance banner in 'Linode' landing page when one or more Linodes get impacted.
timerange-verification.spec.tsCloud Manager Cypress Tests→Integration tests for verifying Cloudpulse custom and preset configurations » should implement and validate custom date/time picker for a specific date and time range
linode-storage.spec.tsCloud Manager Cypress Tests→linode storage tab » delete disk

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/notificationsAndEvents/qemu-reboot-upgrade-notice.spec.ts,cypress/e2e/core/cloudpulse/timerange-verification.spec.ts,cypress/e2e/core/linodes/linode-storage.spec.ts"

@github-project-automation github-project-automation Bot moved this from Review to Merged in Cloud Manager Aug 7, 2025
@corya-akamai
corya-akamai deleted the UIE-9051-block-non-beta-iam branch September 10, 2025 19:48
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.

3 participants