search

OCA / l10n-france

France Localization for Odoo
GNU Affero General Public License v3.0
42 stars 118 forks source link

[14.0] l10n_fr_account_tax_unece: follow changes in l10n_fr #380

Closed alexis-via closed 2 years ago

alexis-via commented 2 years ago

Update module following a big change in l10n_fr https://github.com/odoo/odoo/pull/84918

Several XMLIDs were changed in l10n_fr, which broke this module.

alexis-via commented 2 years ago

/ocabot merge minor

OCA-git-bot commented 2 years ago

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

OCA-git-bot commented 2 years ago

@alexis-via your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-380-by-alexis-via-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.

alexis-via commented 2 years ago

The impact is bigger than I thought, because the tests of l10n_fr_account_vat_return are also broken. I disable those tests temporarily.

alexis-via commented 2 years ago

/ocabot merge minor

OCA-git-bot commented 2 years ago

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

OCA-git-bot commented 2 years ago

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