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] point-of-sale: discount line on ticket #32989

Closed

Conversation

switch87
Copy link
Contributor

Description of the issue/feature this PR addresses:
Due to the use of t-else instead of t-elif an extra line
will be printed on the ticket for every producteven if
not desired.

Current behavior before PR:
An extra line with the price without discount will be
printed even if there is no discount on the product.

Desired behavior after PR is merged:
The line will only be printed if there is a discount.

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

Before this fix an extra line with the price without discount will be
printed even if there is no discount on the product.

After this fix the line will only be printed if there is a discount.

The error occured due to the use of t-else instead of t-elif.
@switch87 switch87 requested a review from nim-odoo April 26, 2019 14:00
@robodoo robodoo added seen 🙂 CI 🤖 Robodoo has seen passing statuses labels Apr 26, 2019
Copy link
Contributor

@nim-odoo nim-odoo left a comment

Choose a reason for hiding this comment

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

robodoo r+

robodoo pushed a commit that referenced this pull request Apr 26, 2019
Before this fix an extra line with the price without discount will be
printed even if there is no discount on the product.

After this fix the line will only be printed if there is a discount.

The error occured due to the use of t-else instead of t-elif.

closes #32989

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
@C3POdoo C3POdoo added the RD research & development, internal work label Apr 26, 2019
@robodoo
Copy link
Contributor

robodoo commented Apr 26, 2019

Merged, thanks!

@robodoo robodoo closed this Apr 26, 2019
@fw-bot fw-bot deleted the saas-12.1-fix-discount-print-gpe branch October 19, 2019 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses Point of Sale RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants