Skip to content

[FIX] pos_meal_voucher: remove unoverloadable function.#565

Merged
OCA-git-bot merged 1 commit intoOCA:12.0from
legalsylvain:12.0-FIX-pos_meal_voucher-circular
May 13, 2021
Merged

[FIX] pos_meal_voucher: remove unoverloadable function.#565
OCA-git-bot merged 1 commit intoOCA:12.0from
legalsylvain:12.0-FIX-pos_meal_voucher-circular

Conversation

@legalsylvain
Copy link
Copy Markdown
Contributor

the function that the PR remove is not overloable with that way. More precisely it fails if there is a confirm action inside due to the poor design of odoo Point of sale.
the same patch should be applied in pos_tare.

Step to reproduce :

  • install pos_tare and pos_meal_voucher.
  • open the PoS and try to confirm an action with wrong values. (negative quantity AND to much meal voucher to receive).
  • it's not possible to confirm the order, even if we click on confirm. (infinite loop of confirmation).

CC : @grap : #/board/144/card/1099 @quentinDupont

CC : @Fkawala

@legalsylvain legalsylvain added this to the 12.0 milestone Nov 16, 2020
@legalsylvain
Copy link
Copy Markdown
Contributor Author

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-565-by-legalsylvain-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 13, 2021
Signed-off-by legalsylvain
@OCA-git-bot
Copy link
Copy Markdown
Contributor

It looks like something changed on 12.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 12.0-ocabot-merge-pr-565-by-legalsylvain-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 4df03ab into OCA:12.0 May 13, 2021
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 6f25d22. 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.

3 participants