Skip to content
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

[16.0][FIX] stock_mts_mto_rule: make route noupdate #2126

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

kevinkhao
Copy link
Contributor

@kevinkhao kevinkhao commented Aug 1, 2024

See #2125
This is just a quick workaround that keeps the initial rule, allows for persistent customization of it and prevents raise on module upgrade if the DB happens to have the wrong configuration case.

@kevinkhao kevinkhao changed the title [FIX] revert nocompany on mto, mts rule, make route noupdate [16.0][FIX] stock_mts_mto_rule: revert nocompany on mto, mts rule, make route noupdate Aug 1, 2024
@kevinkhao kevinkhao force-pushed the 16.0-stock_mts_mto_rule-revert_nocompany branch from 4e8bb9e to 4ef7eb1 Compare August 1, 2024 18:27
@kevinkhao kevinkhao force-pushed the 16.0-stock_mts_mto_rule-revert_nocompany branch from 4ef7eb1 to f790cd9 Compare August 1, 2024 18:41
@kevinkhao kevinkhao changed the title [16.0][FIX] stock_mts_mto_rule: revert nocompany on mto, mts rule, make route noupdate [16.0][FIX] stock_mts_mto_rule: make route noupdate Aug 1, 2024
@kevinkhao kevinkhao marked this pull request as draft August 2, 2024 07:32
@kevinkhao kevinkhao marked this pull request as ready for review August 27, 2024 13:30
@kevinkhao
Copy link
Contributor Author

@florian-dacosta what do you think ?

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