Monday, June 21, 2021
3 changes · 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
This update fixes an internal automated test so it matches recent changes in how the web interface loads menu information. It helps keep quality checks reliable without changing the user experience.
Original PR description
With the rewrite of the webclient in OWL 29731b404fe the load_menus method now returns all menus instead of the root menu. This commit adapts the click_everywhere tests to the change.
A previously skipped automated test for Documents Spreadsheet has been re-enabled after the underlying issue was already fixed. This helps ensure future changes are checked properly without affecting day-to-day users.
Original PR description
The test was skipped in commit 6be4fbb because the test was wrong. But it was fixed in the mean time by ea4c9ec Probably a rebase error :)