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

login and signup api with jwt authentication #124

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

Conversation

Rimaru06
Copy link

@Rimaru06 Rimaru06 commented May 22, 2024

Related issue

#46

Description

i have create a login and signup api using jwt auth
and use zod for validation data
also make a middleware for jwt verification
and bcryptjs for password hashing

Type of PR

  • Feature enhancement

Screenshots

signup api
image
login api
image
for jwt verification made a dummy endpoint
image

Copy link

vercel bot commented May 22, 2024

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

Name Status Preview Comments Updated (UTC)
kaam-do ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 22, 2024 0:05am

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@PatilHarshh
Copy link
Owner

what did @InshiyaRavat do ?

@Rimaru06
Copy link
Author

what did @InshiyaRavat do ?

Made signup api

@InshiyaRavat
Copy link
Contributor

what did @InshiyaRavat do ?

signup api and bcrypt

@Rimaru06
Copy link
Author

@PatilHarshh can you please label this PR . This is necessary and was advised on discord for your reference I have attached ss below -:

image

@PatilHarshh
Copy link
Owner

@InshiyaRavat You should work on UI part of sign up because it seems that you both worked in single PR !!

@Rimaru06
Copy link
Author

@PatilHarshh when will you merge this one ?

@Rimaru06
Copy link
Author

@PatilHarshh @InshiyaRavat told me to work on frontend part she said that she won't be able to contribute further because she is busy

@InshiyaRavat
Copy link
Contributor

@InshiyaRavat You should work on UI part of sign up because it seems that you both worked in single PR !!

@Rimaru06 will be working on the frontend part. As of now I won't be able to complete the task soon.

@PatilHarshh
Copy link
Owner

@Rimaru06 any updates

@Rimaru06
Copy link
Author

Rimaru06 commented Jun 2, 2024

@Rimaru06 any updates

Sorry for the delay my End Term examination is going on that's why i am a little busy.
Can you please merge the backend pr ,
i 'll be doing the frontend part on 7th june because my exam will be over on 6th june

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

Successfully merging this pull request may close these issues.

3 participants