fix: Preserve selected environment

Flagsmith / flagsmith

Open Source Feature Flagging and Remote Config Service. Host on-prem or use our hosted version at https://flagsmith.com/

https://flagsmith.com/

BSD 3-Clause "New" or "Revised" License

4.77k stars 365 forks source link

Closed kyle-ssg closed 3 months ago

kyle-ssg commented 3 months ago

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Fixes 2 issues:

Clicking features in the nav switches the environment to the first in the list
Going to a link like segments then back to features doesn't preserve the last selected environment

Click the features nav item after switching environment, it preserves the same url
Navigate from features to segments then back, it preserves the environment

vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
flagsmith-frontend-preview	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 19, 2024 1:41pm
flagsmith-frontend-staging	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 19, 2024 1:41pm

1 Ignored Deployment

github-actions[bot] commented 3 months ago

Uffizzi Preview deployment-53220 was deleted.

sentry-io[bot] commented 3 months ago

This pull request was deployed and Sentry observed the following issues:

‼️ TypeError: Cannot read properties of null (reading 'getItem') new Promise(<anonymous>) View Issue

_{Did you find this useful? React with a 👍 or 👎}