Integrated fronTend with backend

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

Integrated fronTend with backend #124

Closed tushitshukla03 closed 5 months ago

tushitshukla03 commented 5 months ago

Overview

This PR fixes or fixes part of #[N/A].
This PR does the following: [This Pr made the eventlist dynamic on home page. Integrated frontend with backend.Made the specific page for the event and reintegrated the cart as per ticket]

Essential Checklist

[ ] The PR title starts with "Fix #bugnum: ", followed by a short, clear summary of the changes. (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
[ ] "Allow edits from maintainers" is checked. (See here for instructions on how to enable it.)
[ ] The PR is made from a branch that's not called "main/master".

Proof that changes are correct

PR Pointers

If you need a review or an answer to a question, and don't have permissions to assign people, leave a comment like the following: "{{Question/comment}} @{{reviewer_username}} PTAL".
Never force push. If you do, your PR will be closed.

netlify[bot] commented 5 months ago

Deploy Preview for merch-site canceled.

Name	Link
Latest commit	40c5c0735d2ca26d7e1b78477dd988bf137614b1
Latest deploy log	https://app.netlify.com/sites/merch-site/deploys/65bfece6781d5a00082b9b4c

Priyansh61 commented 5 months ago

Image alligment needs to be fixed!

Priyansh61 commented 5 months ago

checkout functionality is not working, like after adding to cart the checkout doesnt show the price of the item you added to the cart.
There should also be a buy now button along with this because currently only add to cart is visible.
Something wrong with the image ratio at the event list page
This section looks a bit odd currently, Ig some UI changes needs to be done here
The cart logic is not working correclty, this needs to be fixed. IG..

Priyansh61 commented 5 months ago

@tushitshukla03 PTAL!

tushitshukla03 commented 5 months ago

@Priyansh61 Take a look, made some changes

Priyansh61 commented 5 months ago

@tushitshukla03 Almost everything looks but can you please once check the image ratio either they get distorted or either their ratio becomes unusually.

And like on the EventList page can you change the event ratio so that the images looks great. After this we can merge the PR.

Priyansh61 commented 5 months ago

@tushitshukla03 Can we improve this layout? And why are we getting by 3

tushitshukla03 commented 5 months ago

@tushitshukla03 Can we improve this layout? And why are we getting by 3

fixed the image size

Priyansh61 commented 5 months ago

@arminpatel @thejaysharma check and merge the PR

bsoc-bitbyte / GetIt

Integrated fronTend with backend #124

Overview

Essential Checklist

Proof that changes are correct

PR Pointers

✅ Deploy Preview for merch-site canceled.

Deploy Preview for merch-site canceled.