-
-
Notifications
You must be signed in to change notification settings - Fork 5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: preserve # in the URL if the screen hasn't changed - partially done #11876
fix: preserve # in the URL if the screen hasn't changed - partially done #11876
Conversation
✅ Deploy Preview for react-navigation-example ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Also let's add some tests as well. You can add them under |
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #11876 +/- ##
==========================================
+ Coverage 76.99% 77.13% +0.13%
==========================================
Files 207 209 +2
Lines 6151 6193 +42
Branches 2420 2442 +22
==========================================
+ Hits 4736 4777 +41
- Misses 1361 1362 +1
Partials 54 54 ☔ View full report in Codecov by Sentry. |
Motivation
when updating some params on the page, the hash shouldn’t be removed