Skip to content

[15.0][IMP] stock_weighing: Do not break workflow to record weight is user wants to print labels #19

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

Conversation

sergio-teruel
Copy link

cc @Tecnativa TT51483

When user wants record the weight he can click on enter key and the wizard is reloaded but if the print label option is checked the workflow is broken.
We need depend of web_ir_actions_act_multi module to allow print label and return de reload action wizard.

ping @chienandalu @carlosdauden

@sergio-teruel sergio-teruel force-pushed the 15.0-IMP-stock_weighing-mulri-actions-dependency branch from f45c415 to 1ac4d07 Compare October 30, 2024 11:33
Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

👍 Pulling a new dependecy isn't great, but it's needed for the bug fix

@pedrobaeza pedrobaeza added this to the 15.0 milestone Oct 30, 2024
@sergio-teruel sergio-teruel force-pushed the 15.0-IMP-stock_weighing-mulri-actions-dependency branch from 1ac4d07 to e6e6bb1 Compare October 30, 2024 11:55
@sergio-teruel sergio-teruel force-pushed the 15.0-IMP-stock_weighing-mulri-actions-dependency branch from e6e6bb1 to 922c2a5 Compare November 6, 2024 13:32
@pedrobaeza
Copy link
Member

/ocabot merge major

@carlosdauden
Copy link

Looks like the bot was lazy

@pedrobaeza
Copy link
Member

/ocabot merge major

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-19-by-pedrobaeza-bump-major, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit de81c72 into OCA:15.0 Nov 8, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 15.0-IMP-stock_weighing-mulri-actions-dependency branch November 8, 2024 08:25
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.

5 participants