8000 restoreCurrentCart also restore currency by lopes-vincent · Pull Request #3015 · thelia/thelia · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

restoreCurrentCart also restore currency #3015

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

Merged
merged 6 commits into from
Aug 24, 2022

Conversation

lopes-vincent
Copy link
Member

This PR fix #2837 by checking if currency is not null

AnthonyMeedle and others added 6 commits April 23, 2021 15:30
Pour que la devise sélectionnée soit reprise en même temps que le panier (quand on arrive sur le site), sinon on a des prix de produit d'une devise anciennement sélectionné mais le site est sur la devise par défaut.
@lopes-vincent lopes-vincent merged commit 0a9b942 into thelia:master Aug 24, 2022
@lopes-vincent lopes-vincent deleted the AnthonyMeedle-patch-18 branch August 24, 2022 12:24
lopes-vincent added a commit to lopes-vincent/thelia that referenced this pull request Aug 25, 2022
* Update cart.php - restoreCurrentCart

Pour que la devise sélectionnée soit reprise en même temps que le panier (quand on arrive sur le site), sinon on a des prix de produit d'une devise anciennement sélectionné mais le site est sur la devise par défaut.

* Fix docker for 2.4

* Fix when currency is null on cart

Co-authored-by: Chevrier <anthony@meedle.fr>
lopes-vincent added a commit that referenced this pull request Aug 25, 2022
* Update cart.php - restoreCurrentCart

Pour que la devise sélectionnée soit reprise en même temps que le panier (quand on arrive sur le site), sinon on a des prix de produit d'une devise anciennement sélectionné mais le site est sur la devise par défaut.

* Fix docker for 2.4

* Fix when currency is null on cart

Co-authored-by: Chevrier <anthony@meedle.fr>
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.

2 participants
0