search

thepassle / app-tools

128 stars 8 forks source link

fix: browser back after refresh #24

Closed thepassle closed 10 months ago

justinlau12 commented 11 months ago

Hey, I think this PR looks good! I believe it fixes the duplicate entry after page load/refresh since it adds a replace case check upon every render of the router, only thing I'm not sure about is if it will fix the back nav breaking issue I was talking about, but that issue may also similarly be caused by duplicate history entries.

thepassle commented 10 months ago

only thing I'm not sure about is if it will fix the back nav breaking issue I was talking about

I was able to reproduce the issue you reported locally, and this change should fix it. Will release this shortly