Skip to content

Conversation

@dchyrva-akamai
Copy link
Contributor

Description πŸ“

STORIF-62 Has not been fully tested yet and should be removed from the release.

πŸ‘€ 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 tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@mjac0bs mjac0bs changed the base branch from release-v1.147.0 to staging July 28, 2025 14:03
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

πŸŽ‰ 690 passing tests on test run #2 β†—οΈŽ

❌ Failingβœ… Passingβ†ͺ️ SkippedπŸ• Duration
0 Failing690 Passing4 Skipped117m 43s

@mjac0bs mjac0bs marked this pull request as ready for review July 28, 2025 14:53
@mjac0bs mjac0bs requested a review from a team as a code owner July 28, 2025 14:53
@mjac0bs mjac0bs requested review from dwiley-akamai and mjac0bs and removed request for a team July 28, 2025 14:53
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.

@dchyrva-akamai I didn't seem to have write access to this branch to be able to make the changelog revert directly. I can do that on the staging branch once it is merged, but if you have a setting unchecked like the below, please check the checkbox in the future.

Screenshot 2025-07-28 at 7 59 48β€―AM

image

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.

βœ… confirmed banner removed - thank you!

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Jul 28, 2025
@coliu-akamai coliu-akamai added the Approved Multiple approvals and ready to merge! label Jul 28, 2025
@mjac0bs mjac0bs merged commit b0128a1 into linode:staging Jul 28, 2025
35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Jul 28, 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! 🚨 Urgent

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants