Skip to content

refactor: [M3-9388 ] - Move Status Page queries queries#12468

Merged
cpathipa merged 13 commits intolinode:developfrom
cpathipa:M3-9388
Jul 9, 2025
Merged

refactor: [M3-9388 ] - Move Status Page queries queries#12468
cpathipa merged 13 commits intolinode:developfrom
cpathipa:M3-9388

Conversation

@cpathipa
Copy link
Contributor

@cpathipa cpathipa commented Jul 3, 2025

Description 📝

Migrates Status Page queries, as well as all dependencies, under the @linode/queries package.

Changes 🔄

  • Creates new statusPage/ directory in the @linode/queries package with the following structure:
databases/
|__ index.ts
|__ statusPage.ts
|__ keys.ts
|__ requests.ts
  • Updates import paths throughout packages/manager.

How to test 🧪

  • Rely on e2e tests and CI pipeline to ensure changes do not break the application.
  • No changes to the UI as a result of this PR.

Verification steps

  • Verify that the types feature works as expected.
  • Verify that other major flows continue to work as expected.
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

@cpathipa cpathipa self-assigned this Jul 3, 2025
@cpathipa cpathipa requested a review from a team as a code owner July 3, 2025 14:55
@cpathipa cpathipa requested review from harsh-akamai and mjac0bs and removed request for a team July 3, 2025 14:55
@cpathipa cpathipa marked this pull request as draft July 3, 2025 14:55
@cpathipa cpathipa changed the title Add statuspage query refactor: [M3-9388 ] - Move Status Page queries queries Jul 3, 2025
@cpathipa cpathipa marked this pull request as ready for review July 3, 2025 17:54
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.

CI failures are legit.

@cpathipa cpathipa requested a review from mjac0bs July 8, 2025 01:48
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.

Thanks Chandra!

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Jul 8, 2025
@mjac0bs mjac0bs added the Approved Multiple approvals and ready to merge! label Jul 8, 2025
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 675 passing tests on test run #12 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing675 Passing4 Skipped118m 26s

@cpathipa cpathipa merged commit 83ca276 into linode:develop Jul 9, 2025
35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Jul 9, 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! Modularization

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants