OCA / l10n-netherlands

Odoo Dutch Localization
GNU Affero General Public License v3.0
21 stars 55 forks source link

[14.0][FIX] l10n_nl_tax_statement: don't break when writing list of ids on x2many #419

Closed hbrunn closed 2 days ago

Stephanvandenberg commented 1 month ago

Tested on a test instance and this PR works

OCA-git-bot commented 1 month ago

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). 🤖

bosd commented 2 days ago

@gjotten Can you please merge?

gjotten commented 2 days ago

@NL66278 please merge

NL66278 commented 2 days ago

/ocabot merge minor

OCA-git-bot commented 2 days ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-419-by-NL66278-bump-minor, awaiting test results.

OCA-git-bot commented 2 days ago

@NL66278 your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-419-by-NL66278-bump-minor.

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.

NL66278 commented 2 days ago

/ocabot merge minor

OCA-git-bot commented 2 days ago

What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-419-by-NL66278-bump-minor, awaiting test results.

OCA-git-bot commented 2 days ago

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