search

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
32 stars 71 forks source link

[Feature] Removing the static images and storing them in Supabase Storage #152

Closed Yogesh-ok55 closed 5 months ago

Yogesh-ok55 commented 5 months ago

Description

On home page the cart logo shows 8 cart element by default i think it is not as intended and on clicking add to cart it is not being added to cart and nor it have option to delete cart element

Screenshots

Screenshot 2024-05-29 163225 Screenshot 2024-05-29 163425 Screenshot 2024-05-29 163447

Any additional information?

Please assign this issue under GSSOC'24 to me , i would like to work on this to solve this issue and want to add some extra functionality..

What browser are you seeing the problem on?

No response

pooranjoyb commented 5 months ago

Congratulations, @Yogesh-ok55! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

pooranjoyb commented 5 months ago

How would you try to implement that? Coz we also need to store the cart items in Supabase @Yogesh-ok55

Yogesh-ok55 commented 5 months ago

As you have stored images locally so for that I can use useContext() hook to implement this , for managing states or if you want I can also do this using Supabase but the problem here is that you have stored images locally..

pooranjoyb commented 5 months ago

I think we need to create a Cart Slice but yes keeping the images in Supabase Storage is also necessary. Can you work on that? @Yogesh-ok55

Yogesh-ok55 commented 5 months ago

Yes I can do..

pooranjoyb commented 5 months ago

Okay drop me your email here in the comment. Store the images in supabase DB and use them in the frontend. We'll move to other tasks later. @Yogesh-ok55

Yogesh-ok55 commented 5 months ago

yogeshh.ok@gmail.com

pooranjoyb commented 5 months ago

Invite sent for supabase. check your mail. @Yogesh-ok55

pooranjoyb commented 5 months ago

Invite sent for supabase. check your mail. @Yogesh-ok55

image

Yogesh-ok55 commented 5 months ago

Received and joined

pooranjoyb commented 5 months ago

Received and joined

All the best! @Yogesh-ok55

Yogesh-ok55 commented 5 months ago

Sorry for being late , actually it was my end term going on , so I could not able to do this please assign again this to me I have already uploaded images to supabase and it is being rendered from supabase

Yogesh-ok55 commented 5 months ago

image showing this token is invalid.. so please do this again

pooranjoyb commented 5 months ago

it was expired as you were too late :) @Yogesh-ok55

Mahitej28 commented 5 months ago

@Yogesh-ok55 what are the updates? give it by today eod

Yogesh-ok55 commented 5 months ago

It is already done I have also create a PR but I have been asked to fix some build errors which I have done but I don't what next to do

Mahitej28 commented 5 months ago

@Yogesh-ok55 mention your issue number in the pr

Yogesh-ok55 commented 5 months ago

@Yogesh-ok55 mention your issue number in the pr

issue number #152 , PR number 220 , i have made some changes to fix build errors which was problem in my PR , these changes will solve build errors