stripe-samples / checkout-one-time-payments

Use Checkout to quickly collect one-time payments.
https://checkout.stripe.dev/?mode=payment
MIT License
819 stars 439 forks source link

var basicPhotoButton = document.getElementById('basic-photo-button'); #40

Open Int-Soft opened 3 years ago

Int-Soft commented 3 years ago

This Var: var basicPhotoButton = document.getElementById('basic-photo-button'); not in the HTML file. Can I use this project to list more than one item per page? How can we achieve this? ADD more price vars in .env and edit the script.js in client update server.js in server to retrieve the price

Int-Soft commented 3 years ago

Another problem is when you checkout, then cancel the order the return link is to http://your_domain.com:port/canceled.html or https://your_domain.com:port/canceled.html not to https://your_domain.com/canceled.html in case you are using haproxy.

Int-Soft commented 3 years ago

Adding more payment methods in .env like PAYMENT_METHODS="card, alipay" gives error.

Int-Soft commented 3 years ago

Bug with Safari browser on iOS 12.4.8 The price cannot be displayed. https://drive.google.com/file/d/1OqnHijGq9jzwrOUk0fbSEu6e17Opjk6i/view?usp=sharing

cjavilla-stripe commented 3 years ago

Another problem is when you checkout, then cancel the order the return link is to http://your_domain.com:port/canceled.html or https://your_domain.com:port/canceled.html not to https://your_domain.com/canceled.html in case you are using haproxy.

This bit is determined by the value passed for success_url and cancel_url, you can include or exclude the port in the URL. For now the "DOMAIN" environment variable is really used as the base path and includes the port.

cjavilla-stripe commented 3 years ago

Adding more payment methods in .env like PAYMENT_METHODS="card, alipay" gives error.

Hmm. Seems like that environment variable is not being respected by all server languages. Going to create a new separate issue for that.