bank-payment icon indicating copy to clipboard operation
bank-payment copied to clipboard

[FIX] _check_payment_mode_company_constrains

Open mileo opened this issue 5 years ago • 7 comments

Can breaks multi company environments, for example: https://travis-ci.org/github/OCA/l10n-brazil/jobs/722154732#L2375

mileo avatar Sep 08 '20 20:09 mileo

Onchange of odoo/sale: https://github.com/odoo/odoo/blob/12.0/addons/sale/models/sale.py#L477-L490

Is called without company, than this can handle some unexpected behavior at multi company environment.

When the company of the sales order is different from the user who confirms it.

mileo avatar Sep 15 '20 13:09 mileo

/ocabot merge patch

dreispt avatar Jul 03 '21 14:07 dreispt

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 12.0-ocabot-merge-pr-730-by-dreispt-bump-patch, awaiting test results.

OCA-git-bot avatar Jul 03 '21 14:07 OCA-git-bot

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-730-by-dreispt-bump-patch, awaiting test results.

OCA-git-bot avatar Jul 03 '21 18:07 OCA-git-bot

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-730-by-dreispt-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

OCA-git-bot avatar Jul 03 '21 23:07 OCA-git-bot

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Aug 07 '22 12:08 github-actions[bot]

/ocbot merge patch

dreispt avatar Aug 10 '22 21:08 dreispt

/ocabot merge patch

dreispt avatar Oct 01 '22 15:10 dreispt

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 12.0-ocabot-merge-pr-730-by-dreispt-bump-patch, awaiting test results.

OCA-git-bot avatar Oct 01 '22 15:10 OCA-git-bot

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

OCA-git-bot avatar Oct 01 '22 18:10 OCA-git-bot