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

We should'nt display cart discounts on product in side cart #1758 #2247

Merged
merged 5 commits into from
Jan 18, 2019
Merged

We should'nt display cart discounts on product in side cart #1758 #2247

merged 5 commits into from
Jan 18, 2019

Conversation

mcspronko
Copy link
Collaborator

Related issues

We should'nt display cart discounts on product in side cart #1758

Short description and why it's useful

The product totals part of the Product.vue template has been removed. Now, minicart shows product prices as is on the line level.

Screenshots of visual changes before/after (if there are any)

After the fix has been applied:
https://prnt.sc/m8vyst

Screenshot of passed e2e tests (if you are using our standard setup as a backend)

(run yarn test:e2e and paste the results. If you are not using our standard backend setup or demo.vuestorefront.io you can ommit this step)

Upgrade Notes and Changelog

  • No upgrade steps required (100% backward compatibility)
  • I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

Contribution and currently important rules acceptance

@pkarw
Copy link
Collaborator

pkarw commented Jan 18, 2019

Thanks! I believe some of the users might want to display the prices as it was done before - can we add the config.cart.displayItemDiscounts by default (set in default.json) = true for this?

To get the config You can just:

import rootStore from “@vue-storefront/store” and then:

rootStorw.state.config.cart ...

Copy link
Collaborator

@pkarw pkarw left a comment

Choose a reason for hiding this comment

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

This is cool but will be even better if added the config option for setting the price display behavior

@mcspronko
Copy link
Collaborator Author

@pkarw makes sense to add a setting for this, but from a user perspective, seeing total incl tax price on product item level seems like confusing information. What do you think? Maybe we have to just show Price with Incl/Excl Tax on line level?

@mcspronko
Copy link
Collaborator Author

@pkarw this is more like "display totals" vs "display product prices" option.

@pkarw
Copy link
Collaborator

pkarw commented Jan 18, 2019

It's fine!

@pkarw pkarw merged commit d921973 into vuestorefront:develop Jan 18, 2019
@pkarw pkarw added this to the 1.8 milestone Feb 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants