Skip to content

Conversation

@hana-akamai
Copy link
Contributor

Description 📝

Fix selected Geographical Area persistence issue when switching between Core and Distributed Region tabs

Preview 📷

Before After
before.mov
after.mov

How to test 🧪

Prerequisites

(How to setup test environment)

  • Ensure gecko feature flag is enabled and your account has gecko customer tags (reach out for more info)

Reproduction steps

(How to reproduce the issue, if applicable)

  • Go to the Linode Create page on prod or another branch
  • Click on the Distributed Region tab, select a value other than All, click on the Core Region tab, switch back to the Distributed tab
  • You should see the Geographical Area changes back to All but the Regions are filtered based off the previous area selection

Verification steps

(How to verify changes)

  • Checkout this branch or preview link and go to the Linode Create page
  • Click on the Distributed Region tab, select a value other than All, click on the Core Region tab, switch back to the Distributed tab
  • You should see the Geographical Area persist the value you selected before switching and the Regions are still filtered accordingly
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

@hana-akamai hana-akamai self-assigned this Jul 25, 2025
@hana-akamai hana-akamai added the Bug Fixes for regressions or bugs label Jul 25, 2025
@hana-akamai hana-akamai marked this pull request as ready for review July 25, 2025 19:57
@hana-akamai hana-akamai requested a review from a team as a code owner July 25, 2025 19:57
@hana-akamai hana-akamai requested review from bnussman-akamai and dwiley-akamai and removed request for a team July 25, 2025 19:57
@hana-akamai hana-akamai added the Gecko LA Relating to Gecko LA label Jul 25, 2025
Copy link
Member

@bnussman-akamai bnussman-akamai left a comment

Choose a reason for hiding this comment

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

Looks good! Just left some comments with test improvements/clean up

Copy link
Contributor

@dwiley-akamai dwiley-akamai left a comment

Choose a reason for hiding this comment

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

Fix confirmed & no regressions observed ✅
Code review ✅
Checks pass ✅

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

Cloud Manager UI test results

🎉 690 passing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing690 Passing4 Skipped122m 47s

@hana-akamai hana-akamai merged commit ef3a4e4 into linode:develop Jul 28, 2025
35 checks passed
@hana-akamai hana-akamai deleted the M3-9986-distributed-region-geo-area-select-bug branch July 28, 2025 18:53
@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! Bug Fixes for regressions or bugs Gecko LA Relating to Gecko LA

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants