upcoming: [UIE 9401] – Implement Share Groups Landing page tabs#13471
upcoming: [UIE 9401] – Implement Share Groups Landing page tabs#13471fabrice-akamai merged 16 commits intolinode:developfrom
Conversation
pmakode-akamai
left a comment
There was a problem hiding this comment.
thanks @fabrice-akamai! This looks good to me 👍 Could we also add a Upcoming Features changeset for this PR?
| Membership Requests is coming soon... | ||
| </Notice> | ||
| )} | ||
| </SafeTabPanel> |
There was a problem hiding this comment.
Why are we not using routed tabs here? See other examples with lazy routes + Outlet
There was a problem hiding this comment.
Do we also need the Outlet placeholder here? I saw that it was implemented by the parent component ImagesLandingV2, but looking at the corresponding ImageLibraryTabs, I noticed that the SafeTabPanels are rendered directly in the file without using Outlet. Can you confirm if I need to add it to both files?
There was a problem hiding this comment.
Ideally yes, since the use of SafeTabPanels should be deprecated (need to do a PR for that). This could however be a follow up if you'd like to keep the PR's scope narrower
There was a problem hiding this comment.
Sure, I can create a follow-up ticket to replace the use of SafeTabPanels so we can merge this PR if it's non-blocking.
dwiley-akamai
left a comment
There was a problem hiding this comment.
Verification steps & tab functionality ✅
Will do another pass of the PR once the routed tabs have been implemented
packages/manager/src/features/Images/ImagesLanding/v2/ShareGroups/ShareGroupsTabs.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Purvesh Makode <pmakode@akamai.com>
Cloud Manager UI test results🔺 4 failing tests on test run #7 ↗︎
Details
TroubleshootingUse this command to re-run the failing tests: pnpm cy:run -s "cypress/e2e/core/parentChild/account-switching.spec.ts,cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts" |
||||||||||||||||||||||||||
dwiley-akamai
left a comment
There was a problem hiding this comment.
Approving as the routed tabs will be implemented in a follow-up PR.
packages/manager/.changeset/pr-13471-upcoming-features-1773085243886.md
Outdated
Show resolved
Hide resolved
…243886.md Co-authored-by: Dajahi Wiley <114682940+dwiley-akamai@users.noreply.github.com>
…de#13471) * Added missing padding around the Managed dashboard card * changed spacing to spacingFunction * Implement share groups landing page tabs * Update image subtab interface * Fix image utils test case * Update packages/manager/src/features/Images/utils.test.tsx Co-authored-by: Purvesh Makode <pmakode@akamai.com> * Added changeset: Add share groups tabs * Update file name and consolidate imports * Rename file for consistency * Update packages/manager/.changeset/pr-13471-upcoming-features-1773085243886.md Co-authored-by: Dajahi Wiley <114682940+dwiley-akamai@users.noreply.github.com> --------- Co-authored-by: Purvesh Makode <pmakode@akamai.com> Co-authored-by: Dajahi Wiley <114682940+dwiley-akamai@users.noreply.github.com>
Description 📝
This PR implements the Share Groups Landing page and its corresponding subtabs:
This feature builds upon the existing work on the Image Library tabs and lays the groundwork for the Share groups implementation under the
isPrivateImageSharingEnablefeature flag.Changes 🔄
List any change(s) relevant to the reviewer.
ShareGroupsTabs.tsxandShareGroupsTabsConfig.tsxgetImageLibrarySubTabIndexto be more generic so the Share Groups tab can reuse the same logicShareGroupsTabs.tsxScope 🚢
Upon production release, changes in this PR will be visible to:
Target release date 🗓️
Please specify a release date (and environment, if applicable) to guarantee timely review of this PR. If exact date is not known, please approximate and update it as needed.
Preview 📷
How to test 🧪
Prerequisites
(How to setup test environment)
isPrivateImageSharingEnabledis enabled for testing the feature under the newImagespageReproduction steps
Verification steps
(How to verify changes)
localhost:3000/images/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 ✅