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

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

Open
Int-Soft opened this issue Oct 2, 2020 · 5 comments
Open

Comments

@Int-Soft
Copy link

Int-Soft commented Oct 2, 2020

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
Copy link
Author

Int-Soft commented Oct 2, 2020

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
Copy link
Author

Int-Soft commented Oct 2, 2020

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

@Int-Soft
Copy link
Author

Int-Soft commented Oct 3, 2020

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
Copy link
Contributor

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
Copy link
Contributor

cjavilla-stripe commented Oct 7, 2020

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants