Skip to content
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-9268] – Migrate ErrorState to ui package #11718

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

harsh-akamai
Copy link
Contributor

Description πŸ“

Migrate Divider to ui package

How to test πŸ§ͺ

Verification steps

  • All tests should pass
  • There should be no adverse impacts observed in Cloud
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

@harsh-akamai harsh-akamai marked this pull request as ready for review February 25, 2025 08:13
@harsh-akamai harsh-akamai requested a review from a team as a code owner February 25, 2025 08:13
@harsh-akamai harsh-akamai requested review from bnussman-akamai and coliu-akamai and removed request for a team February 25, 2025 08:13
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

πŸŽ‰ 523 passing tests on test run #4 β†—οΈŽ

❌ Failingβœ… Passingβ†ͺ️ SkippedπŸ• Duration
0 Failing523 Passing3 Skipped106m 3s

Copy link
Contributor

@coliu-akamai coliu-akamai left a comment

Choose a reason for hiding this comment

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

Thanks @harsh-akamai! looks like there's some merge conflicts that need to be resolved. We should remove the ErrorState component from the manager package too and then add a changeset to the Manager package underneath the Removed category

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants