-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
🚀 Feature: Add login/signup page to connect with backend auth APIs of nodejs #19
Comments
hello there! I would like to work on this please. |
Okay @techy4shri , you can start working. |
I can help you with it in this. |
@mehul-m-prajapati Just want to confirm if you still want me to raise the PR or not. Let me know when you can. |
@techy4shri ; yes, you can work on it when you have time. |
i'm working on this... |
check it out for any required changes |
@parassareen4 : Add login option in navbar |
sure, on it in a while. |
Sorry I was out, just got things back. Let me know if you need any help or I will continue to explore other issues in the meantime? |
@parassareen4 are you working on the backend too? |
@techy4shri as you were absent @parassareen4 was assigned & is already working on this issue, you can try out other issues. |
Sure. I just got my account back after suspension so yeah, it's been a while. |
no problem you are free to take on the pending issues, share your approach with @mehul-m-prajapati and get assigned on the issue |
added the login button in navbar how you want the routing structure at client site to be? |
@techy4shri : you can look at other issues. |
is there anything else to do in this issue? |
🔖 Feature description
Login/SignUp with NodeJS
Use passport for auth
Take username, email, password as signup details
Add Login/SignUp pages
Make request to backend using Axios
Notify status of login/signup using toaster messages in frontend
NOTE: Follow proper coding structure when creating backend code
🎤 Why is this feature needed ?
No response
✌️ How do you aim to achieve this?
No response
🔄️ Additional Information
Email and username should be unique
There should be a validation check for email, username for signup
Implement best solution which is used in real-world
Skeleton code is done in PR Added backend code #46
👀 Have you spent some time to check if this feature request has been raised before?
The text was updated successfully, but these errors were encountered: