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

Added Google OAuth #271

Closed masterboy376 closed 4 days ago

masterboy376 commented 1 week ago

Added Google OAuth using google cloud and Supabase. Admin is advised to configure Google OAuth on google cloud and Supabase after merging this PR.

Fixes #265

Type of change

Please give a X on it which is applicable

Screenshots and Videos:

screen-recorder-mon-jun-24-2024-18-18-33.webm

Checklist:

Give a X on, which is applicable

vercel[bot] commented 1 week 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 28, 2024 2:30am
masterboy376 commented 1 week ago

@pooranjoyb PTAL

pooranjoyb commented 1 week ago

@pooranjoyb PTAL

show me the profile. how is the user created in supabase??? @masterboy376

masterboy376 commented 1 week ago

@pooranjoyb, here is the screenshot Screenshot (172)

pooranjoyb commented 1 week ago

yeah so you can see that we need to add something like not-found and in the edit profile make sure that we can add these not-found details

masterboy376 commented 1 week ago

@pooranjoyb PTAL Screenshot (173)

masterboy376 commented 6 days ago

@pooranjoyb PTAL

pooranjoyb commented 5 days ago

@pooranjoyb PTAL

Not working. Please check, something might be wrong :/

screen-recorder-thu-jun-27-2024-22-31-39.webm

masterboy376 commented 5 days ago

Hello @pooranjoyb This was happening due to domain mismatch in supabase redirect urls. Fixed it. Now login is working on both vercel deployments as well as local servers. screen-recorder-fri-jun-28-2024-07-50-58.webm

masterboy376 commented 5 days ago

@pooranjoyb PTAL

pooranjoyb commented 4 days ago

@pooranjoyb PTAL

OK will review and update accordingly.

pooranjoyb commented 4 days ago

Hey @masterboy376 Do let me the environment variables I need to add for this feature. Otherwise everything looks good for the merge.

masterboy376 commented 4 days ago

@pooranjoyb there is no additional env variable needs to be added

masterboy376 commented 4 days ago

@pooranjoyb However I have created the Oauth consent screen using my personal google cloud account. So, when ever you have time, do create a consent screen using organization's account.

pooranjoyb commented 4 days ago

@pooranjoyb However I have created the Oauth consent screen using my personal google cloud account. So, when ever you have time, do create a consent screen using organization's account.

Yes, I do need to set this up. Drop me the steps below.