Skip to content
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

[ADD] l10n_fr_oca: FORK of l10n_fr + reactivate tests in l10n_fr_account_vat_return + TVA millesime 2023 + Selenium IDE in separate module #420

Merged
merged 5 commits into from
Mar 4, 2023

Conversation

alexis-via
Copy link
Contributor

For the reasons of the fork of l10n_fr, read #379

Re-enable tests in l10n_fr_account_vat_return (tests had to be disabled following the "horror PR" on l10n_fr on October 6th 2022 odoo/odoo#84918)

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

@alexis-via alexis-via changed the title [ADD] l10n_fr_oca: FORK of l10n_fr + reactivate tests in l10n_fr_account_vat_return [ADD] l10n_fr_oca: FORK of l10n_fr + reactivate tests in l10n_fr_account_vat_return + TVA millesime 2023 Feb 20, 2023
@alexis-via alexis-via changed the title [ADD] l10n_fr_oca: FORK of l10n_fr + reactivate tests in l10n_fr_account_vat_return + TVA millesime 2023 [ADD] l10n_fr_oca: FORK of l10n_fr + reactivate tests in l10n_fr_account_vat_return + TVA millesime 2023 + Selenium IDE in separate module Feb 20, 2023
@alexis-via
Copy link
Contributor Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

OCA-git-bot added a commit that referenced this pull request Mar 2, 2023
Signed-off-by alexis-via
@OCA-git-bot
Copy link
Contributor

@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-420-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
Copy link
Contributor Author

GRRRR, pre-commit fails and is completely unrelated

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
Move Selenium IDE support from l10n_fr_account_vat_return to a new
decidated module l10n_fr_account_vat_return_selenium.
This feature is beta, so I prefer to move it to a separate module.
Allows to see the partner of the source invoice directly from the log line.
@alexis-via alexis-via force-pushed the 14-add-l10n_fr_oca branch from 9c5966a to b536d1c Compare March 4, 2023 13:39
@alexis-via
Copy link
Contributor Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 00f9b4a into OCA:14.0 Mar 4, 2023
@OCA-git-bot
Copy link
Contributor

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

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.

3 participants