Skip to content

Conversation

@aaleksee-akamai
Copy link
Contributor

Description 📝

Remove the toggle in the Add A User drawer, by default, the user will have limited access

Changes 🔄

List any change(s) relevant to the reviewer.

  • remove toggle

Target release date 🗓️

(dev)

Preview 📷

Include a screenshot or screen recording of the change.

🔒 Use the Mask Sensitive Data setting for security.

💡 Use <video src="" /> tag when including recordings in table.

Before After
image image

How to test 🧪

Prerequisites

(How to setup test environment)

  • Ensure the Identity and Access Beta flag is enabled in dev tools
  • Click on the Add A User button

Reproduction steps

Verification steps

(How to verify changes)

  • there is no toggle in the drawer
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 unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@github-actions
Copy link

github-actions bot commented Mar 18, 2025

Coverage Report:
Base Coverage: 79.96%
Current Coverage: 79.96%

@cpathipa cpathipa requested review from cpathipa and removed request for harsh-akamai March 18, 2025 13:53
@cpathipa cpathipa added the Add'tl Approval Needed Waiting on another approval! label Mar 18, 2025
@aaleksee-akamai aaleksee-akamai force-pushed the UIE-8604-drawer-create-user-remove-toggle branch from 14d8f80 to 232fba1 Compare March 18, 2025 14:28
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing538 Passing3 Skipped137m 49s

Details

Failing Tests
SpecTest
open-support-ticket.spec.tsopen support tickets » can create a support ticket with an entity

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/helpAndSupport/open-support-ticket.spec.ts"

Copy link
Contributor

@mjac0bs mjac0bs left a comment

Choose a reason for hiding this comment

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

Thanks @aaleksee-akamai - confirmed the change and I left two small bits of feedback.

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Mar 18, 2025
@mjac0bs mjac0bs added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Mar 18, 2025
@aaleksee-akamai aaleksee-akamai force-pushed the UIE-8604-drawer-create-user-remove-toggle branch from 232fba1 to 0594cc3 Compare March 18, 2025 16:29
@cpathipa cpathipa merged commit 9bf7c34 into linode:develop Mar 18, 2025
26 of 27 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge! IAM (Identity & Access Management)

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants