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

[14.0][FIX] account_operating_unit Backport payment patches from v15 #737

Merged
merged 7 commits into from
Jan 28, 2025

Conversation

aleuffre
Copy link
Contributor

@aleuffre aleuffre commented Jan 28, 2025

Backport of multiple patches from v15 related to handling payments

2612956
07dced0
20eb5b2
b449ed0
fcc3363
8157960
31cd3b5

ps-tubtim and others added 7 commits January 28, 2025 15:23
…al has no operating unit no matter if OU are self balanced
… operating units.

That was possible in old versions <v12 but it was removed for unknown
reason.

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.
@aleuffre aleuffre changed the title 14.0 backport patches [14.0][FIX] Backport payment patches from v15 Jan 28, 2025
Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Thanks

@AaronHForgeFlow AaronHForgeFlow changed the title [14.0][FIX] Backport payment patches from v15 [14.0][FIX] account_operating_unit Backport payment patches from v15 Jan 28, 2025
Copy link

@francesco-ooops francesco-ooops left a comment

Choose a reason for hiding this comment

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

Functional ok!

Copy link

@GSLabIt GSLabIt left a comment

Choose a reason for hiding this comment

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

code LGTM

@francesco-ooops
Copy link

@AaronHForgeFlow could this be merged? Thanks!

@AaronHForgeFlow
Copy link
Contributor

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-737-by-AaronHForgeFlow-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 9203d20 into OCA:14.0 Jan 28, 2025
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

8 participants