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

Things wrong! #12

Open
Gabriel7553 opened this issue May 28, 2021 · 1 comment
Open

Things wrong! #12

Gabriel7553 opened this issue May 28, 2021 · 1 comment

Comments

@Gabriel7553
Copy link

  1. You can still order without having anything in the cart!(Just get something in stock and place the order(since we don't even need to pay yet) like the camera and click the go back arrow to go back to the order screen, and since we reset the cart for each order, the cart will be empty, and we can spam the place order button filling the database with orders with no products and this follows the second issue.)

  2. Why do place the order and we don't even pay yet? The user can just keep placing the order filling up all the database and when someone pays then what lol. Shouldn't the paypal buttons be in the placeorder.js screen and not the orderscreen.js. So this problem doesn't occur and we only see the paid orders. I'm struggling right now as I'm a beginner so I need to learn more just to fix this. I feel like it's easy but when I try I face many problems.

3.Many other problems as well that need to be fixed. For example on your twitter saying you found some problems and a hacker changed the images. I don't know if you had updated the course and showed a way but yeah seems pretty dumb. I'm still going to try to learn more and fix these problems myself or get some help but yeah why do it like that in the first place.

@khalidfasely
Copy link

I got the same problem and I solve the first one: you could see the pull request (#33)

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