bsoc-bitbyte / GetIt

Streamlined college e-commerce: Simplifying merch buying and selling for students.
https://getit.iiitdmj.ac.in/
17 stars 51 forks source link

feat: add to Cart #93

Closed AkashPaloju closed 1 year ago

AkashPaloju commented 1 year ago

Overview

  1. This PR fixes or fixes part of #90 .
  2. This PR does the following: makes the 'Add to Cart' and 'Remove from cart(Cross X)' button functional. The Cart page now displays the items added to the cart by the user by retrieving data from Vuex store. The Vuex store is persisted across browser refreshes/sessions as it is saved in the local storage of the browser. The User is notified about adding/removing products through informative "toasts".

Essential Checklist

Proof that changes are correct

https://github.com/bsoc-bitbyte/GetIt/assets/119884665/81c0edea-c130-4771-a187-a15461501db4

PR Pointers

@tushitshukla03 PTAL, please let me know if any further adjustments are required

netlify[bot] commented 1 year ago

Deploy Preview for merch-site ready!

Name Link
Latest commit d907902a31f079e0a25626cd79ea7d7cb90b55b6
Latest deploy log https://app.netlify.com/sites/merch-site/deploys/64bb9179f4da38000846e342
Deploy Preview https://deploy-preview-93--merch-site.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

tushitshukla03 commented 1 year ago

@AkashPaloju First sync your branch and implement + - feature of cart and also make the heading and text in cart fixed at top. Change the toast position to bottom center and also make it square.

AkashPaloju commented 1 year ago

@tushitshukla03 someone removed the product heading in the product page, is it requested by the maintainers or removed accidentally? should i add the title which was like earlier or leave it

image
tushitshukla03 commented 1 year ago

@tushitshukla03 someone removed the product heading in the product page, is it requested by the maintainers or removed accidentally? should i add the title which was like earlier or leave it

image

No it was removed as the title in the mobile view was two times.

AkashPaloju commented 1 year ago

@tushitshukla03 I am closing this PR since I have some issues with the current branch. I am opening a new PR