Skip to content

Conversation

@vvrossem
Copy link
Contributor

@vvrossem vvrossem commented Jan 13, 2026

account.move should default on the journal's operating unit if set instead of the user's operating unit.

Note: default should not be combined with compute.

`account.move` should default on the journal's operating unit (if set)
instead of the user's operating unit
@vvrossem vvrossem changed the title [IMP] account_operating_unit: account.move::_default_operating_unit_id [17.0][IMP] account_operating_unit: account.move::_default_operating_unit_id Jan 13, 2026
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.

👍 Makes sense

Copy link

@ivantodorovich ivantodorovich left a comment

Choose a reason for hiding this comment

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

LG

@AaronHForgeFlow
Copy link
Contributor

/ocabot merge patch

Could you please check if FW port is needed?

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-825-by-AaronHForgeFlow-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 22ec06e into OCA:17.0 Jan 14, 2026
7 of 9 checks passed
@OCA-git-bot
Copy link
Contributor

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

@vvrossem
Copy link
Contributor Author

@AaronHForgeFlow
Copy link
Contributor

@vvrossem Thank you!

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.

4 participants