Skip to content

[FIX] website: do not redirect after editing menus from the builder #4766

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

Open
wants to merge 1 commit into
base: master-mysterious-egg-next
Choose a base branch
from

Conversation

robinlej
Copy link

Steps to reproduce:

  • Edit a page
  • Click on a menu item and click on the "Edit menu" icon
  • Add an item and save => The iframe is reloaded, but leads to a 404 instead of staying on the same page.

This is because when passing a URL to reloadEditor, the given string is then encoded with encodeURIComponent, which will append the string to the current domain. We can just leave the parameter empty.

@robodoo
Copy link

robodoo commented May 21, 2025

This PR targets the un-managed branch odoo-dev/odoo:master-mysterious-egg-next, it needs to be retargeted before it can be merged.

@ged-odoo ged-odoo force-pushed the master-mysterious-egg-next branch from 6e36b56 to 48edd7d Compare May 22, 2025 07:31
@robinlej robinlej force-pushed the master-mysterious-egg-role-7 branch from 8837af6 to 3f350ff Compare May 22, 2025 09:23
Steps to reproduce:
- Edit a page
- Click on a menu item and click on the "Edit menu" icon
- Add an item and save
=> The iframe is reloaded, but leads to a 404 instead of staying on the
same page.

This is because when passing a URL to `reloadEditor`, the given string
is then encoded with `encodeURIComponent`, which will append the string
to the current domain. We can just leave the parameter empty.
@robinlej robinlej force-pushed the master-mysterious-egg-role-7 branch from 8265a2b to b022c00 Compare May 23, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants