-
-
Notifications
You must be signed in to change notification settings - Fork 238
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
[16.0][MIG] account_operating_unit #656
[16.0][MIG] account_operating_unit #656
Conversation
… view in report. Other minor fixes.
…f setting an inter-OU accounts triggers from other modules when creating companies
* Add management of payments using operating units * Trial Balance by operating units * Improve tests
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: operating-unit-15.0/operating-unit-15.0-account_operating_unit Translate-URL: https://translation.odoo-community.org/projects/operating-unit-15-0/operating-unit-15-0-account_operating_unit/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
To be also included: #685 Thanks |
|
with different 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.
…rpart base line This commit adds the operating unit to the cash basis counterpart base line when cash basis accounting is enabled.
inspired by v17 migration
5579a49
to
f0c0e9c
Compare
@AaronHForgeFlow picked the commits you requested and added the patch for the widget |
@hbrunn Thank you! I will do a functional review. |
/ocabot migration account_operating_unit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functional review, it works as expected. Thanks
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 7b751c5. Thanks a lot for contributing to OCA. ❤️ |
TODO
The module is already perfectly functional, but I don't consider the migration to be properly done without this