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

[14.0][FIX] pos_pricelist_show_discount: remove fragile demo data #1064

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

aleuffre
Copy link

Removing pricelist and pos configuration from the demo data because it creates issues when combined with l10n modules that modify the company's currency, due to inconsistencies in currencies.

The same demo data is now created programmatically during tests instead.

Removing pricelist and pos configuration from the demo data
because it creates issues when combined with l10n modules that
modify the company's currency, due to inconsistencies in currencies.

The same demo data is now created programmatically during tests instead.
@OCA-git-bot
Copy link
Contributor

Hi @pytech-bot,
some modules you are maintaining are being modified, check this out!

@francesco-ooops
Copy link
Contributor

@GSLabIt what do you think?

@aleuffre aleuffre changed the title [FIX] pos_pricelist_show_discount: remove fragile demo data [14.0][FIX] pos_pricelist_show_discount: remove fragile demo data Oct 13, 2023
@aleuffre
Copy link
Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-1064-by-aleuffre-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit fe04a70 into OCA:14.0 Oct 13, 2023
@OCA-git-bot
Copy link
Contributor

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

@aleuffre aleuffre deleted the 14.0-pos-discount-no-demo-data branch October 31, 2023 08:36
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.

4 participants