Add checkout page#536
Conversation
dmytro-kashcheiev
left a comment
There was a problem hiding this comment.
@Vladyslav-Kuksiuk its ok now from the code perspective, but I want to request a few more features from UI-UX:
- Since VAT ID is a toxic field, I propose to add an in-progress spinner somewhere in the field that shows that we are actually validating it right now and it's not either aprooved / rejected.
- IF VAT passes verification, highlight the field a little green and even maybe small text like:
Provided ID is active and valid. - Lets fix the following behavior: When I have already filled everything and then press
Continue to payment. If on a new page, I decided I want to double check my address and press the return button. I lose part of the filled data and proceed back.
|
@Vladyslav-Kuksiuk General comment. Let’s make the product ID a path variable instead of an argument. The current issue is that a customer could effectively specify any ID, and our payment system would accept it even if it isn’t the expected Spine product. If we specify products as path variables—like I believe this approach has another advantage: logically, we’ll have a separate page for each product, which makes sense for a static site. @dmytro-kashcheiev what do you think? |
|
@Vladyslav-Kuksiuk OK. I agree. |
dmytro-kashcheiev
left a comment
There was a problem hiding this comment.
@Vladyslav-Kuksiuk LGTM from my side.
MykytaPimonovTD
left a comment
There was a problem hiding this comment.
@Vladyslav-Kuksiuk LGTM.
This PR adds checkout page
.../checkout?product={product-id}.Predefined test Paygate product:
spine-standard-support.Page view:
Server Error view (when something is broken in the middle of checkout):
Product not found page (when Paygate is available, but answers 404):
Thank you page (
.../checkout-completed)Ooops page