Skip to content

[19.0][FIX] web_m2x_options#3559

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
qrtl:6740-fix-web_m2x_option
May 29, 2026
Merged

[19.0][FIX] web_m2x_options#3559
OCA-git-bot merged 1 commit into
OCA:19.0from
qrtl:6740-fix-web_m2x_option

Conversation

@AungKoKoLin1997
Copy link
Copy Markdown
Contributor

@AungKoKoLin1997 AungKoKoLin1997 commented May 27, 2026

Related: #3550

This PR enforces create/create_edit/open on the shared Many2One component so specialized widgets (eg. sol_product_many2one) honor the options too.

@qrtl QT6740

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 6740-fix-web_m2x_option branch from 79d1ca3 to 5ff1dbf Compare May 27, 2026 06:35
@AungKoKoLin1997 AungKoKoLin1997 changed the title [FIX] web_m2x_options [19.0][FIX] web_m2x_options May 27, 2026
@AungKoKoLin1997 AungKoKoLin1997 marked this pull request as ready for review May 27, 2026 06:38
Copy link
Copy Markdown

@kanda999 kanda999 left a comment

Choose a reason for hiding this comment

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

I confirmed that the setting also works for sale.order.line when web_m2x_options.create: False is set in the system parameters.

Image

@AungKoKoLin1997
Copy link
Copy Markdown
Contributor Author

@etobella Can you please review the PR and merge if you don't have any concern?
This is the follow-up fix for #3382.

Copy link
Copy Markdown
Member

@etobella etobella left a comment

Choose a reason for hiding this comment

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

Makes sense to me

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 19.0-ocabot-merge-pr-3559-by-etobella-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 632341c into OCA:19.0 May 29, 2026
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 1c9f002. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants