Skip to content

Conversation

@ihor-sviziev
Copy link
Contributor

@ihor-sviziev ihor-sviziev commented Jan 16, 2018

Description

The discount label was not being shown in the shopping cart totals description
Continuing of #11883

Fixed Issues

  1. Discount Rule does not show Default Rule Label #11497: Discount Rule does not show Default Rule Label
  2. Cart Price Rule Label is not working #11428: Cart Price Rule Label is not working

Manual testing scenarios

  1. Add a discount to quote
  2. Go to the shopping cart and now see the discount label in the totals description

Related PRs

#13141
#13223

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

Fix js static test

(cherry picked from commit 3fc1f40)
Fix js static test

(cherry picked from commit 0f36e0e)
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Jan 16, 2018

CLA assistant check
All committers have signed the CLA.

@ihor-sviziev ihor-sviziev changed the base branch from 2.2-develop to 2.3-develop January 16, 2018 15:17
@magento-engcom-team magento-engcom-team added Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release labels Jan 16, 2018
@ishakhsuvarov ishakhsuvarov self-assigned this Jan 17, 2018
@ishakhsuvarov ishakhsuvarov added this to the January 2018 milestone Jan 17, 2018
@magento-team magento-team merged commit 968d7af into magento:2.3-develop Jan 18, 2018
@magento-engcom-team magento-engcom-team added Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line labels Jan 18, 2018
@raymond62
Copy link

This seems to work out on most cases but it is not for nl_NL.

1) I have Magento 2.3 installed with a en_GB, de_DE and a nl_NL Store. Following the Order Process in en_GB and de_DE is working fine, no issues. The issue appears only in nl_NL on all browsers on pc and mobile devices. I have tried flush, clean, rm all static files, del cookies ...

2) Following the Order Process it shows following (in stead of the expected discount label text it shows "%s"):

grafik

3) When i change the language to de_DE or en_GB, the expected discount label text appears in nl_NL, what also is wrong. I expect it in the choosen language.
(in this case it should also be "VAT (19%)" and not "BTW (19%)". Those are set in the Tax Area. Seems to be another bug...):

grafik

4) Going back to nl_NL Store the "%s" is back again.

5) Case it helps, here additionally my Store Discount Labels that should appear on the side

grafik

Thank you for your assistance, i appreciate a lot

@ihor-sviziev
Copy link
Contributor Author

Hi @raymond62,
Could you create separate issue for that?

@ihor-sviziev ihor-sviziev deleted the fix-discount-label-2.3 branch April 2, 2018 09:56
@raymond62
Copy link

Thanks, opened new Issue #14494

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fixed in 2.2.x The issue has been fixed in 2.2 release line Fixed in 2.3.x The issue has been fixed in 2.3 release line Progress: accept Release Line: 2.3 Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants