-
-
Notifications
You must be signed in to change notification settings - Fork 118
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] l10n_fr_oca #423
Conversation
3805dc5
to
f507b0f
Compare
/ocabot migration l10n_fr_oca |
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
@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-423-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 your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-423-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. |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
@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-423-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. |
Re-enable tests in l10n_fr_account_vat_return Tests in l10n_fr_account_vat_return now use the chart of account of l10n_fr_oca, which reduce a lot the number of lines of code needed for the initial setup of the tests
b5a518b
to
4e3a1e9
Compare
Remove tax codes because we can't create them as in l10n_fr because the code must be unique and we don't want this module to depend on l10n_fr. And this module is designed for the OCA VAT return, so we don't need them.
4e3a1e9
to
46e7f25
Compare
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 83b71aa. Thanks a lot for contributing to OCA. ❤️ |
No description provided.