Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Designed and Coded Register Page #68

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

kashnx
Copy link

@kashnx kashnx commented Jun 1, 2024

Designed and coded register page at localhost:3000/register.

Screenshot 2024-06-01 235721

Copy link

vercel bot commented Jun 1, 2024

@kashnx is attempting to deploy a commit to the punyakrit's projects Team on Vercel.

A member of the Team first needs to authorize it.

@punyakrit
Copy link
Owner

There is no backend logic for saving user data as we are currently doing with google provider

@kashnx
Copy link
Author

kashnx commented Jun 2, 2024

There is no backend logic for saving user data as we are currently doing with google provider

I'm a frontend developer and i proposed to design the frontend for it. Please merge my pr.

@punyakrit
Copy link
Owner

punyakrit commented Jun 2, 2024

cant merge then without having full effect as currently having google login

@kashnx
Copy link
Author

kashnx commented Jun 2, 2024

cant merge then without having full effect as currently having google login

It wont be affected as the page is on localhost:3000/register. You can open an issue for the backend.

@punyakrit
Copy link
Owner

still you can give a try and learn how backend works in nextjs

@kashnx
Copy link
Author

kashnx commented Jun 3, 2024

still you can give a try and learn how backend works in nextjs

I can try, but if you are willing to assign level3 to this issue.

@punyakrit
Copy link
Owner

if it is you are able to add register with email and password and below it set with google provider and their simple backend with nextauth and full logic then would provide level3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants