fix: [UIE-10308] - IAM Parent/Child - SwitchAccount Drawer: hide search if no child account#13464
Conversation
… if no child account
mpolotsk-akamai
left a comment
There was a problem hiding this comment.
Confirmed the verification steps. LGTM ✅
|
@bnussman-akamai I raised this exact concern in the ticket - you can check the convo there. UX ruled in favor of the approach in this PR, which I also initially disagreed with. Not sure this is worth arguing a lot more over but including @tzmiivsk-akamai in this convo in case there's reconsideration after another concern is raised. |
|
I’m still convinced that removing extra information here will reduce cognitive load and visual noise. For the empty state, we don’t need the search and the description. |
|
That sounds good to me. Thanks @abailly-akamai & @tzmiivsk-akamai |
Cloud Manager UI test results🔺 3 failing tests on test run #5 ↗︎
Details
TroubleshootingUse this command to re-run the failing tests: pnpm cy:run -s "cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts,cypress/e2e/core/parentChild/account-switching.spec.ts" |
|||||||||||||||||||||||


Description 📝
Small PR to hide the search bar in the
SwitchAccountDrawerif there no child account to search forChanges 🔄
List any change(s) relevant to the reviewer.
SwitchAccountDrawerif there no child account to search forScope 🚢
Upon production release, changes in this PR will be visible to:
Preview 📷
How to test 🧪
Prerequisites
An IAM parent/child account
Reproduction steps
Verification steps
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
As an Author, before moving this PR from Draft to Open, I confirmed ✅