8000 [15.0][FIX] web_m2x_options M2ODialog not fully functional by theangryangel · Pull Request #2304 · OCA/web · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[15.0][FIX] web_m2x_options M2ODialog not fully functional #2304

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

Closed
wants to merge 1 commit into from

Conversation

theangryangel
Copy link
Member

Prior to this patch:

  • M2ODialog does not function correctly due to upstream removal of search_create_popup custom_event (odoo/odoo@1d4d2a6). The Create and Edit button just does "nothing".
  • M2ODialog offers both Create and Create and Edit, even if the options are disabled

Copy link
Contributor
@CRogos CRogos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Code is working and looks good.

I am not sure if it is really useful to also remove the "Create and Edit" button in the popup with the same configuration keys.
image

We removed the "Create..." from the dropdown because it was used accidently to many times, but using the button would be ok for us.

I did another fix #2318 addressing another issue, which will have some conflict on the _search changes.

Shall we add another config key for the popup and merge both PR before merging it into 15.0?

@theangryangel
Copy link
Member Author

We removed the "Create..." from the dropdown because it was used accidently to many times, but using the button would be ok for us.

Happy for it to be another option, but the client who raised this correctly pointed out that if you're banning create on the dropdown then it almost certainly makes no sense to be in the popup, because you'll still end up with a badly configured product :)

However, I do understand that not everyone will have this opinion, so I'm happy to make it a separate option, if that's what people want? :)

Shall we add another config key for the popup and merge both PR before merging it into 15.0?

Makes sense, but I'm not sure if I have the time to do this for at least a few days (probably longer). Do you have time to try and pull these changes into yours, and I close this PR? What works for you? :)

…stream removal of search_create_popup custom_event, M2ODialog offers both Create and Create and Edit, even if the options are diabled
@theangryangel theangryangel force-pushed the 15.0-web_m2x_options-T4679 branch from 52bde74 to 8f14788 Compare March 8, 2023 13:30
@theangryangel
Copy link
Member Author

Closing as superseded by #2901

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
0