-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
- Added product remove from stock after success pay. #103
base: master
Are you sure you want to change the base?
Conversation
- Added Public / protected route to update only product stock after purchase. - Fixed Product list not updating after purchase. - Deleted CART_CLEAR_ITEMS completely in order to correctly update the stock from each of the products (in case they are more than 1 in the same cart.)
your code format is not aligned with the repo. you have used a semicolon at the end of each command. fix this issue and send a new commit. |
Fixed @basir. Sorry, for the obvious mistake. Hope you find it useful! |
@Pablets I tried to implement your fix, but found it insecure. |
@agt-ru Thanks for the advice, I'll try to fix it as soon as possible. |
@Pablets , did you finished working on the security thing? |
@aashiqahmed98 No, I thougth in a solution but didn't have time to implement it yet. The route need to be avalaible only when certain conditions are meet in order to be secure. It needs some state modeling before do anything. |
@Pablets Can you help me out to integrate stripe? |
@Pablets Can you help me integrate a feature? I'm new to react, and need help integrating this |
@Pablets did you finished implementation on the security thing? |
@Pablets Did you find a way to allow to add more qty to the same product from product screen |
@Pablets I can't get this to work. Can you send me a PR? |
Issue #102 fix