Skip to content

Conversation

@nikhagra-akamai
Copy link
Contributor

Description 📝

upcoming: [DI-25355] - Added util function to filter region based on monitor capability

Changes 🔄

List any change(s) relevant to the reviewer.

  1. Added a function filterRegionByServiceType in alerts/utils/utils
  2. Remove supportedRegionIds from CloudPulseResourceTypeMapFlag

Target release date 🗓️

12th August

Preview 📷

Note

No visual change is available for this PR

How to test 🧪

Note

you can verify these changes by comparing to the UI from develop branch as there is no visual modified with these changes

  1. Switch to mock user
  2. Go to metrics tab
  3. Select a dashboard and then open region filter
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 tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@mjac0bs mjac0bs changed the title Region support upcoming: [DI-25355] - Added util function to filter region based on monitor capability Jul 24, 2025
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.

thank you!

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Jul 24, 2025
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 690 passing tests on test run #5 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing690 Passing4 Skipped123m 30s

@nikhagra-akamai nikhagra-akamai merged commit b12096d into linode:develop Jul 25, 2025
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants