-
Notifications
You must be signed in to change notification settings - Fork 4
Bug with French toggling not in prod #437
Comments
For the record, I was able to reproduce this locally on master as well, so it seems to not be specific to my PR. Will keep looking into it though, just wanted to check that I wasn't letting a bug thru 🤞 |
It's because the route we use for the Start Over link that clears the session data is a unilingual link: When you use the browser back button, it doesn't actually change your language back, it just renders the previous page from cache. So when you hit Should be an easy fix - just create multiple clear routes (in the global controller). I'd leave the existing one, but add Then again, now that I think of it, maybe even easier - the Start route also clears the session. Rather than having the Start Over link use the |
Something that IS happening on prod:
Is this a separate bug? |
oh yeah, so because our Back button is just hooking into the browser's history stack, this is similarly expected behaviour - it's like if you're on any page, hit the Français link, then hit browser back, you'll be looking at an English page, because that's the last thing in your history. Our back link works the exact same way. hmmm... we've got a longstanding item to revisit how the back link works. I'm not sure what the solution will be. We hooked into the browser history stack because it's the easiest implementation - the browser always knows where you came from. The challenge in implementing a back button in our case, is we're not always sure where the user came from - could be different depending on answers to previous questions... but it's something we do want to figure out. |
When I was testing a deployment for another PR, I found the re-emergence of a previously fixed bug. Something weird is happening with toggling to French and then going back to English and later it remembering the previous French request. Here's what I did:
Here's what happened:
Start page appears in French.
I tried this in prod and there was no problem.
The text was updated successfully, but these errors were encountered: