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

[17.0][MIG] pos_hide_cost_price_and_margin: Migration to 17.0 #1185

Merged

Conversation

trisdoan
Copy link
Contributor

@trisdoan trisdoan commented Apr 29, 2024

No description provided.

@trisdoan trisdoan force-pushed the 17.0-mig-pos_hide_cost_price_and_margin branch from bd0d4c4 to 60e4942 Compare May 2, 2024 08:01
hailangvn and others added 13 commits May 6, 2024 00:16
This module is to hide cost, margin, and replenishment info from product
detail in pos. It utilize a setting from pos_margin module but doesn't
need to depend on the later module thanks to the mechanism that if there
is no field iface_display_margin, the value of t-if is False.
Currently translated at 100.0% (1 of 1 strings)

Translation: pos-15.0/pos-15.0-pos_hide_cost_price_and_margin
Translate-URL: https://translation.odoo-community.org/projects/pos-15-0/pos-15-0-pos_hide_cost_price_and_margin/it/
Currently translated at 100.0% (1 of 1 strings)

Translation: pos-15.0/pos-15.0-pos_hide_cost_price_and_margin
Translate-URL: https://translation.odoo-community.org/projects/pos-15-0/pos-15-0-pos_hide_cost_price_and_margin/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: pos-15.0/pos-15.0-pos_hide_cost_price_and_margin
Translate-URL: https://translation.odoo-community.org/projects/pos-15-0/pos-15-0-pos_hide_cost_price_and_margin/
@trisdoan trisdoan force-pushed the 17.0-mig-pos_hide_cost_price_and_margin branch from 60e4942 to 1f6fffd Compare May 5, 2024 17:16
@legalsylvain
Copy link
Contributor

/ocabot migration pos_hide_cost_price_and_margin

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Jul 2, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Jul 2, 2024
30 tasks
Copy link
Member

@fclementic2c fclementic2c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@le-goff-benoit le-goff-benoit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK !

@legalsylvain
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 0543505 into OCA:17.0 Oct 1, 2024
4 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at a7c2dfb. 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.

10 participants