added `orders` while user checkout their cart and remove cart items a…

[dhruv8433]

…fter checkout

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.

@pooranjoyb

[Feature] Make Payment option functionals #233

Fixes #233 (issue)

Type of change

Please give a X on it which is applicable

• [ ] Bug fix (non-breaking change which fixes an issue)
• [x] New feature (non-breaking change which adds functionality)
• [ ] Refactor Code
• [ ] A documentation update
• [ ] Others(mentioned in the issue number)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Test A Describe here

• go to checkout and see supabase orders table

Test B Describe here (if Requred)

Screenshorts and Vedios:

https://github.com/pooranjoyb/popShop/assets/114583978/f0585ac3-9245-422c-84ec-4d5a3fdea808

give screenshorts and vedio of the changes you made

Checklist:

give a X on it which is applicable

• [x] My code follows the style guidelines of this project
• [x] I have performed a self-review of my code
• [x] I have commented my code, particularly in hard-to-understand areas
• [x] I have made corresponding changes to the documentation
• [x] My changes generate no new warnings
• [x] I have added tests that prove my fix is effective or that my feature works
• [x] New and existing unit tests pass locally with my changes
• [x] Any dependent changes have been merged and published in downstream modules

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
pop-shop	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 15, 2024 10:49am

[dhruv8433]

please inform me if you want any modifications

[pooranjoyb]

Replace the Close button with Continue Shopping with the my-green button. This button should redirect user to /home/shop @dhruv8433

Also i can see that you are calling the method to update cart quantity and price on each quantity update. This should not be done as this leads to recurrent requests to the db. Optimize this to make sure that you make only 1 api call to update the quantity and price when user clicks on Place Order.

Remove this quantity from here too. lets not keep it here. Doesn't make sense to allow user to update quantity everywhere.

[dhruv8433]

@pooranjoyb

1. bg-my green

2. remove quantity from product detailed

[pooranjoyb]

Did you do this? @dhruv8433

Also i can see that you are calling the method to update cart quantity and price on each quantity update. This should not be done as this leads to recurrent requests to the db. Optimize this to make sure that you make only 1 api call to update the quantity and price when user clicks on Place Order.

[dhruv8433]

@pooranjoyb

i just removed quantity buttons from detailed product now it's only in cart

[pooranjoyb]

Update this API request wala thing. Read the tasks.

[dhruv8433]

ok got it

[dhruv8433]

@pooranjoyb

now you can check this,

• i updated code now when user change quantity it only set in local redux store
• when usesr click on place order than only it send to supabase and also set their quantity in that

after checkout or place order

[dhruv8433]

@pooranjoyb

please check this and merge it

[pooranjoyb]

There are still some bugs, please cross check the deployment. These are the issues that i've found :

• Sometime, even after i click checkout i cannot see the updated total summary.
• quantity is changing but not updating. (I guess go with the previous flow. that'll do.)

I suggest you to revert the changes. this api call wala, keep this aside for a moment.

[dhruv8433]

@pooranjoyb

i revert the changes

[dhruv8433]

@pooranjoyb

please check this and inform me if you need any modifications

[pooranjoyb]

Cool! awesome work @dhruv8433 🚀