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: display confirmation page returning from redirect after checkout #1227

Merged
merged 1 commit into from
Jul 28, 2022

Conversation

SGrueber
Copy link
Collaborator

@SGrueber SGrueber commented Jul 27, 2022

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?

If the user pays with a redirect after checkout payment method (e.g. online pay) he is routed to an empty basket page after returning from the redirect of the external payment provider.

Issue Number: Closes #

What Is the New Behavior?

He is routed to the order confirmation page after returns from redirect after checkout.

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

This issue occurred because the checkout guard expects a basket, but it doesn't exist after order creation. The guard has been removed from checkout receipt page.

AB#78502

@SGrueber SGrueber requested a review from Eisie96 July 27, 2022 14:34
@SGrueber SGrueber self-assigned this Jul 27, 2022
@SGrueber SGrueber removed the request for review from Eisie96 July 28, 2022 08:09
@SGrueber SGrueber added the bug Something isn't working label Jul 28, 2022
@SGrueber SGrueber added this to the 3.0 milestone Jul 28, 2022
@SGrueber SGrueber requested a review from MaxKless July 28, 2022 08:56
@SGrueber SGrueber merged commit 3ea3b8c into develop Jul 28, 2022
@SGrueber SGrueber deleted the bugfix/confirmation_for_redirect_after_checkout branch July 28, 2022 09:13
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.

2 participants