Monday, June 21, 2021
1 change · master
Resolved issues and error corrections
This fix ensures that legacy client actions correctly update the browser URL with the right context and parameters. It helps users land on and share the correct page state, especially in Discuss, reducing navigation confusion.
Original PR description
Before this commit, the push state from client actions to the URL did not work properly: 1. the action.params key was ignored 2. some "reserved" keys (active_id) were not handled properly 3. within a legacy client action, the call to parent.do_push_state did not work After this commit, all of those work properly letting discuss in particular have the right URL It is worth noting that in the full wowl stack, that the right way to push something in the URL from a component is always to call the router service's pushState when the component is mounted