operating-unit icon indicating copy to clipboard operation
operating-unit copied to clipboard
verified publisher icon OCA

[15.0][IMP] account_operating_unit: allow to pay multiple invoices with different operating units

Open AaronHForgeFlow opened this issue 1 year ago • 2 comments

That was possible in old versions <v12 (https://github.com/OCA/operating-unit/blob/12.0/account_operating_unit/models/account_payment.py#L71) but it was removed for unknown reason when migrating to v13.

In general, reconciling a bank journal entry with an invoice will require Inter OU balancing entries, as long one OU is paying the balance for the other.

If needed I can add a company parameter for restricting paying invoices for different OU if you want to keep compatibility with the actual code.

@Saran440 @alan196 @ForgeFlow

AaronHForgeFlow avatar Apr 24 '24 15:04 AaronHForgeFlow

@GasparJarsa check this PR

alan196 avatar May 07 '24 18:05 alan196

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot avatar May 07 '24 19:05 OCA-git-bot

/ocabot merge patch

AaronHForgeFlow avatar May 22 '24 16:05 AaronHForgeFlow

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 15.0-ocabot-merge-pr-658-by-AaronHForgeFlow-bump-patch, awaiting test results.

OCA-git-bot avatar May 22 '24 16:05 OCA-git-bot

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

OCA-git-bot avatar May 22 '24 16:05 OCA-git-bot