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: add basket ship-to-address to product prices call #1362

Merged
merged 1 commit into from
Jan 30, 2023

Conversation

SGrueber
Copy link
Collaborator

@SGrueber SGrueber commented Jan 27, 2023

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

What Is the Current Behavior?

The product price call respects the customer's default shipping address for the (gross) price display, but if the user changes his basket ship-to-address to a country that has different taxes the prices shown on the cart page might differ from the prices on the product pages.

Issue Number: Closes #

What Is the New Behavior?

The customer's basket ship-to-address is added as a query parameter to the product price REST call to make the result more precise and avoid differences between prices on product pages and cart prices.

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

AB#81275

AB#82962

@SGrueber SGrueber self-assigned this Jan 27, 2023
@SGrueber SGrueber added the bug Something isn't working label Jan 27, 2023
@SGrueber SGrueber added this to the 3.3 milestone Jan 27, 2023
@SGrueber SGrueber requested a review from Eisie96 January 27, 2023 11:33
@sgrumbachIntershop
Copy link

Looks good.

@SGrueber SGrueber merged commit 0e3c633 into develop Jan 30, 2023
@SGrueber SGrueber deleted the bugfix/refine_product_prices_call branch January 30, 2023 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants