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

[FIX] sale_margin: keep old Margin label #43130

Closed
wants to merge 1 commit into from

Conversation

@Feyensv
Copy link
Contributor

Feyensv commented Jan 10, 2020

And use "Margin (%)" for the recently added margin_percent field
computing the margin in % and not in money.

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo robodoo added the seen 🙂 label Jan 10, 2020
@C3POdoo C3POdoo added the RD label Jan 10, 2020
@robodoo robodoo added the CI 🤖 label Jan 10, 2020
And use "Margin (%)" for the recently added margin_percent field
computing the margin in % and not in money.
@Feyensv Feyensv force-pushed the odoo-dev:master-fixes-vfe branch from 223d222 to b10c9d9 Jan 14, 2020
@robodoo robodoo removed the CI 🤖 label Jan 14, 2020
@Feyensv

This comment has been minimized.

Copy link
Contributor Author

Feyensv commented Jan 14, 2020

robodoo pushed a commit that referenced this pull request Jan 14, 2020
And use "Margin (%)" for the recently added margin_percent field
computing the margin in % and not in money.

closes #43130

Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>
@robodoo robodoo closed this Jan 14, 2020
@robodoo robodoo deployed to merge Jan 14, 2020 Active
@Feyensv Feyensv deleted the odoo-dev:master-fixes-vfe branch Jan 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.