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.

Sign up for GitHub

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

Jump to bottom

[Feature]: implement checkout middleware and fix payment error #82

Closed
1 of 3 tasks
Baroshem opened this issue Sep 6, 2021 · 0 comments · Fixed by #89
Closed
1 of 3 tasks

[Feature]: implement checkout middleware and fix payment error #82

Baroshem opened this issue Sep 6, 2021 · 0 comments · Fixed by #89
Assignees
@Baroshem
Labels
enhancement New feature or request

Comments

@Baroshem
Copy link
Contributor

Baroshem commented Sep 6, 2021

How the project can be improved?

Currently we do not have checkout middleware so client can go to customer and then click to move to payment directly and it will result in a wrong flow of order. Also We will have a transition to thank you even thought the process was not handled correctly.

image

What are the acceptance criteria?

  • Create checkout middleware that will allow user to move further in the checkout process only when the required data has been added
  • Handle order payment error similarly to how it was handled when there was existing customer set for order.

Additional information

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@Baroshem Baroshem added the enhancement New feature or request label Sep 6, 2021
@Baroshem Baroshem added this to the 1.0.0-beta.2 milestone Sep 6, 2021
@Baroshem Baroshem self-assigned this Sep 12, 2021
Baroshem added a commit that referenced this issue Sep 12, 2021
@Baroshem
feat: #82 add checkout middleware
c496c64
Baroshem added a commit that referenced this issue Sep 12, 2021
@Baroshem
feat: #82 update changelog and docs
bf64e70
Baroshem added a commit that referenced this issue Sep 12, 2021
@Baroshem
feat: #82 run lint
e5a6845
@Baroshem Baroshem mentioned this issue Sep 12, 2021
Merged
9 tasks
Baroshem added a commit that referenced this issue Sep 14, 2021
@Baroshem
feat: #82 add CheckoutSteps enum
d3c0ea3
Baroshem added a commit that referenced this issue Sep 14, 2021
@Baroshem
Merge pull request #89 from vuestorefront/feat-#82/checkout-middleware
da02273 
feat: #82/checkout middleware
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Baroshem Baroshem

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: #82/checkout middleware vuestorefront-community/vendure
1 participant
@Baroshem