pooranjoyb / popShop

eCommerce Shoping Platform (with Admin Panel) based on ReactTS & Daisy UI, integrating Supabase as BaaS
https://pop-shop-github.vercel.app/
MIT License
29 stars 68 forks source link

update cart and when user increase quantity the price also should be … #192

Closed dhruv8433 closed 1 month ago

dhruv8433 commented 1 month ago

@pooranjoyb

…increased [BUG] update price when we increase quantity in cart #171

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.

Fixes #171

Type of change

Please give a X on it which is applicable

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

Test B Describe here (if Requred)

Screenshorts and Vedios:

https://github.com/pooranjoyb/popShop/assets/114583978/c1493b69-ed4c-424a-ab45-14d01052b11e

Checklist:

give a X on it which is applicable

vercel[bot] commented 1 month ago

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

Name Status Preview Comments Updated (UTC)
pop-shop ❌ Failed (Inspect) Jun 4, 2024 5:24pm
dhruv8433 commented 1 month ago

@pooranjoyb

please add level label and merge it

pooranjoyb commented 1 month ago

Hii @dhruv8433 I was expecting a PR for #109 This feature is not on the priority list. Also doesn't need DB access. I added you on team for #109 Please do that first.

Also there are many build errors. I'd suggest to work on this later. image

pooranjoyb commented 1 month ago

You even mentioned it here https://github.com/pooranjoyb/popShop/issues/171#issuecomment-2146940329

dhruv8433 commented 1 month ago

@pooranjoyb

Sorry i think i misunderstand I think you said #109 merged

Screenshot_20240604_234751_GitHub.jpg

dhruv8433 commented 1 month ago

I will completed #109 till tommorrow

pooranjoyb commented 1 month ago

No issues, but yeah #109 is priority! @dhruv8433