Trisha-tech / OnlineBookSales

This project named Online Book Sales - Backend deals with developing an e-commerce website for Online Book Sale. It provides the user with a catalog of different books available for purchase in the store. In order to facilitate online purchase, a shopping cart is provided to the user.
https://book4u-j5au.onrender.com/
MIT License
137 stars 238 forks source link

Enhanced Functionality and Styling of Cart Page #587

Closed Bhum-ika closed 3 weeks ago

Bhum-ika commented 1 month ago

Description

A clear and concise description of what the PR does.

Fixes #547

Screenshots (if applicable)

Screenshot 2024-10-22 130743 Screenshot 2024-10-22 132416

https://github.com/user-attachments/assets/5870fff8-3b3c-4325-9d3f-9552bb5b2200

This is the dummy content I created to test my changes, it still has increase n decrease functionality in quantity

Additional Context

Kindly add all the tags while merging PR. Thank you for assigning me this issue. If you liked my contribution, kindly upgrade my level because I Not only enhanced UI but worked on functionality too.

Checklist

Make sure to check off all the items before submitting. Mark with [x] if done.

github-actions[bot] commented 1 month ago

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Suman373 commented 1 month ago

@Bhum-ika nice work 👍 ! Are you sure your changes don't generate any new warnings ? ( you haven't filled the checkbox in the PR template)

Bhum-ika commented 1 month ago

@Suman373 It will not generate new warnings, I have checked the tickbox now. Could you please add level 3 here. I worked hard on this. I would really appreciate it

Suman373 commented 1 month ago

@Suman373 It will not generate new warnings, I have checked the tickbox now. Could you please add level 3 here. I worked hard on this. I would really appreciate it

Thank you ! The issue itself was labelled level2. Yes there were a lot of changes. @Trisha-tech may decide if it needs to be upgraded to level3 or not.

Bhum-ika commented 1 month ago

@Trisha-tech Please see

Bhum-ika commented 1 month ago

@Trisha-tech Kindly reply

Suman373 commented 4 weeks ago

@Trisha-tech Kindly reply

Hey @Bhum-ika ! The PA might be busy and she will merge them once she gets time. Till then I'd suggest you to look into other issues.