Skip to content

Conversation

@shamoo53
Copy link
Contributor

📌 Pull Request Title

Description

i implemented:
New Entities:
EmailVerification: Stores email verification tokens
PasswordReset: Stores password reset tokens
New DTOs:
VerifyEmailDto and RequestEmailVerificationDto
ForgotPasswordDto and ResetPasswordDto
New Service:
EmailService: Handles sending verification and reset emails
Enhanced AuthService with new methods:
requestEmailVerification: Generates and sends verification email
verifyEmail: Verifies email token
forgotPassword: Initiates password reset
resetPassword: Resets password with token
New Endpoints in AuthController:
POST /auth/request-verification: Request email verification
POST /auth/verify-email: Verify email with token
POST /auth/forgot-password: Request password reset
POST /auth/reset-password: Reset password with token
Security Features:
Token expiration handling
Token reuse prevention
Secure password hashing
Email verification status tracking
Rate limiting (through NestJS guards)

Related Issues

Changes Made

  • List key changes made in this PR.
  • created New Entities:
  • Enhanced AuthService with new methods:
  • New Endpoints in AuthController:

How to Test

Screenshots (if applicable)

Checklist

  • My code follows the project's coding style.
  • I have tested these changes locally.
  • Documentation has been updated where necessary.

Copy link
Contributor

@portableDD portableDD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good job

please resolve conflict

@shamoo53 shamoo53 requested a review from portableDD May 31, 2025 12:11
Copy link
Contributor

@portableDD portableDD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@portableDD portableDD merged commit b8145dc into Nexacore-Org:main Jun 2, 2025
2 of 3 checks passed
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