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 66 forks source link

dynamic image rendering using supabase #220

Closed Yogesh-ok55 closed 3 weeks ago

Yogesh-ok55 commented 3 weeks ago

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 # (152)

Type of change

using supabase images is being rendered from dynamically before it was statically stored in folder , additionally i have also added Loader in case of any delay

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 ->i have tested it many times to see its efficiency and it is working well and fast in case of delay i have also added loader

Checklist:

give a X on it which is applicable

vercel[bot] commented 3 weeks ago

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

Name Status Preview Comments Updated (UTC)
pop-shop ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 11, 2024 9:54am
pooranjoyb commented 3 weeks ago

Fix the build errors. Run npm run build @Yogesh-ok55

Yogesh-ok55 commented 3 weeks ago

Fix the build errors. Run npm run build @Yogesh-ok55

using npm run build i have fixed all build errors now what should i do

Mahitej28 commented 3 weeks ago

@Yogesh-ok55 there are still some build errors check again, the deployment has failed

Yogesh-ok55 commented 3 weeks ago

i have made some recent changes please consider them...

pooranjoyb commented 3 weeks ago

Hey @Yogesh-ok55 what i can see is you are creating a table to store the images and fetching just the link and rendering them. I guess this is not how it is supposed to work. You were supposed to use the supabase storage for it.

Check the docs for supabase storage, we need to create a bucket to store the images. please let us know if you aren't comfortable with this. We will reassign accordingly.

pooranjoyb commented 3 weeks ago

Do let us know by eod if you wanna continue with this. @Yogesh-ok55

pooranjoyb commented 3 weeks ago

Closing as no updates.