search

OCA / account-financial-tools

Odoo Accountant Financial Tools and Utils
GNU Affero General Public License v3.0
303 stars 749 forks source link

[16.0] account_netting: fix and improve strings #1770

Closed alexis-via closed 9 months ago

alexis-via commented 9 months ago

/ocabot merge nobump

OCA-git-bot commented 9 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-1770-by-alexis-via-bump-nobump, awaiting test results.

OCA-git-bot commented 9 months ago

@alexis-via your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1770-by-alexis-via-bump-nobump.

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.

alexis-via commented 9 months ago

Failure linked to VIES. Let's try a second time

alexis-via commented 9 months ago

/ocabot merge nobump

OCA-git-bot commented 9 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-1770-by-alexis-via-bump-nobump, awaiting test results.

OCA-git-bot commented 9 months ago

@alexis-via your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-1770-by-alexis-via-bump-nobump.

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.

alexis-via commented 9 months ago

It seems that several tests fail, not just VIES. My attempt at fixing the first failing test here: https://github.com/OCA/account-financial-tools/pull/1772

alexis-via commented 9 months ago

/ocabot merge nobump

alexis-via commented 9 months ago

/ocabot merge nobump

OCA-git-bot commented 9 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-1770-by-alexis-via-bump-nobump, awaiting test results.

OCA-git-bot commented 9 months ago

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