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

[Feature] Add more fields to `Auth/Signup` #148

Closed pooranjoyb closed 5 months ago

pooranjoyb commented 5 months ago

Description

As of now we are saving only username, email, and password to the Supabase table. Hence we need to save more data to reflect them back in the User Profile

Tasks

Screenshots

The following is the supabase table you'd need to work upon. The column names can be found in the Screenshot below.

image

Checklist

me-shweta commented 5 months ago

@pooranjoyb Hi! I don't have any experience in this field, it's my first time contributing to a code issue. However, I'm willing to give it a try. If you could help me with some doubts, I'd be able to proceed with this issue.

pooranjoyb commented 5 months ago

Go ahead! @me-shweta 🚀

sristy17 commented 5 months ago

Can you please assign me this, I have the code ready!

me-shweta commented 5 months ago

Hi @pooranjoyb, As part of the recent updates to your authentication system, I've added new fields to the user profile such as first name, last name, gender, phone, address, and account creation date. To ensure that this information is properly stored and accessible, I'd like to confirm where the Supabase table for user profiles is located and who manages its schema. If the table needs to be updated to include these new fields, please let me know the process for making those changes.

pooranjoyb commented 5 months ago

Hi @pooranjoyb, As part of the recent updates to your authentication system, I've added new fields to the user profile such as first name, last name, gender, phone, address, and account creation date. To ensure that this information is properly stored and accessible, I'd like to confirm where the Supabase table for user profiles is located and who manages its schema. If the table needs to be updated to include these new fields, please let me know the process for making those changes.

Drop me your email. I'll give you supabase access 👍🏼

me-shweta commented 5 months ago

done!

pooranjoyb commented 5 months ago

done!

Please mention it here in the comments :) @me-shweta

me-shweta commented 5 months ago

shwetameena0511@gmail.com @pooranjoyb (:

pooranjoyb commented 5 months ago

shwetameena0511@gmail.com

image

pooranjoyb commented 5 months ago

What's the update @me-shweta ?

me-shweta commented 5 months ago

Hi @pooranjoyb,

Tasks Completed

While I have successfully updated the schema and the form, I am currently facing challenges with saving the new data to Supabase during the signup process.

pooranjoyb commented 5 months ago

What are the challenged. Check the response that you receive after hitting the API. That'll help you to identify the issue :) @me-shweta Also it seems like you didn't check the supabase table. Your invite is expired :/

me-shweta commented 5 months ago

@pooranjoyb I need your help regarding GGSOC. I have many successful PRs, and you merged one as well, but my name is still not on the leaderboard. I'm wondering why this happened. Can you help me with this?

pooranjoyb commented 5 months ago

@pooranjoyb I need your help regarding GGSOC. I have many successful PRs, and you merged one as well, but my name is still not on the leaderboard. I'm wondering why this happened. Can you help me with this?

Seems like they just deployed the Leaderboard. Please check after sometime, possibly a server issue on their end. @me-shweta Also do lmk if you still wanna continue with this or wanna drop it and try with other issues?

me-shweta commented 5 months ago

@pooranjoyb I have completed the tasks mentioned above. If you need those files, I will create a PR. Otherwise, I am not sure about the second part.

pooranjoyb commented 5 months ago

@sristy17 Do you wanna continue with the remaining part? we can split this issue in 2 parts.

Sounds Good? @me-shweta @sristy17

sristy17 commented 5 months ago

@sristy17 Do you wanna continue with the remaining part? we can split this issue in 2 parts.

  • @me-shweta raises a PR for task1
  • @sristy17 can finish up task2 and also fetch the logged-in UserData and render them in Profile.

Sounds Good? @me-shweta @sristy17

Yes.

pooranjoyb commented 5 months ago

Okay then @me-shweta create a PR for this issue. We'll review and merge if everything's alright. Meanwhile I'm creating another issue for @sristy17 that's dependent on this issue.

Please drop a PR by today itself @me-shweta

me-shweta commented 5 months ago

sure!

me-shweta commented 5 months ago

@pooranjoyb I've performed the pull request. Please review and let me know if there are any issues. There have been some recent changes in the last 1-2 days, so conflicts may arise since the fork was created. If any conflicts occur, feel free to notify me. :)