-
Notifications
You must be signed in to change notification settings - Fork 367
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: [M3-6910] - Replace instances in : Object Storage #11456
Merged
hasyed-akamai
merged 3 commits into
linode:develop
from
hasyed-akamai:feature/M3-6910-replace-select-with-autocomplete-in-object-storage
Jan 7, 2025
Merged
refactor: [M3-6910] - Replace instances in : Object Storage #11456
hasyed-akamai
merged 3 commits into
linode:develop
from
hasyed-akamai:feature/M3-6910-replace-select-with-autocomplete-in-object-storage
Jan 7, 2025
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
harsh-akamai
requested review from
hana-akamai and
abailly-akamai
and removed request for
a team
January 6, 2025 08:58
hana-akamai
approved these changes
Jan 7, 2025
bnussman-akamai
requested changes
Jan 7, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some unnecessary vi.mocks that we can remove.
packages/manager/src/features/ObjectStorage/BucketDetail/AccessSelect.test.tsx
Outdated
Show resolved
Hide resolved
packages/manager/src/features/ObjectStorage/BucketDetail/ObjectDetailsDrawer.test.tsx
Outdated
Show resolved
Hide resolved
packages/manager/src/features/ObjectStorage/BucketLanding/ClusterSelect.test.tsx
Outdated
Show resolved
Hide resolved
packages/manager/src/features/ObjectStorage/BucketLanding/CreateBucketDrawer.test.tsx
Outdated
Show resolved
Hide resolved
Coverage Report: ✅ |
bnussman-akamai
approved these changes
Jan 7, 2025
Cloud Manager UI test results🎉 469 passing tests on test run #4 ↗︎
|
Cloud Manager E2E Run #7046
Run Properties:
|
Project |
Cloud Manager E2E
|
Branch Review |
develop
|
Run status |
Failed #7046
|
Run duration | 28m 58s |
Commit |
2562a2ff08: refactor: [M3-6910] - Replace instances in : Object Storage (#11456)
|
Committer | hasyed-akamai |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
1
|
Flaky |
0
|
Pending |
2
|
Skipped |
0
|
Passing |
477
|
View all changes introduced in this branch ↗︎ |
Tests for review
cypress/e2e/core/linodes/clone-linode.spec.ts • 1 failed test
Test | Artifacts | |
---|---|---|
clone linode > can clone a Linode from Linode details page |
Screenshots
Video
|
This was referenced Jan 8, 2025
dmcintyr-akamai
pushed a commit
to dmcintyr-akamai/manager
that referenced
this pull request
Jan 9, 2025
…1456) * refactor: [M3-6910] - Replace instances in : Object Storage * Added changeset: Replace Select with Autocomplete component in Object Storage * Remove unnecessary `vi.mocks`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add'tl Approval Needed
Waiting on another approval!
Object Storage
deals with Object Storage
Ready for Review
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description 📝
We want to get rid of our dependency on react-select for accessibility reasons and to consolidate our usage of third-party libraries.
Changes 🔄
Select
with Autocomplete inObject Storage
component.Target release date 🗓️
N/A
How to test 🧪
Verification steps
(How to verify changes)
Autocomplete
.As an Author I have considered 🤔
Check all that apply