Skip to content

Conversation

@rakshityadav1868
Copy link
Contributor

fixes #334

Overview

This PR implements a complete email verification system for user signup, ensuring users verify their email before accessing the platform.

Changes Made

  1. AuthController.js
  • Added sendVerificationEmail() method to send verification emails
  • Updated signUp() to send verification email after account creation
  • Updated signIn() to check email verification status and block unverified users
  1. Created a new page for verification.

  2. SignUpView.vue

  • Changed post-signup redirect from /admin to /verify-email
  • Users now see verification page instead of immediate dashboard access
  • Google OAuth users auto-verified and redirect to dashboard
  1. Auth.js
  • Updated signup() action
  • Updated signin() action to:
    • Show warning toast for unverified email attempts
  1. public.js
  • Added new route /verify-email/:email? for the verification page

Successful signup with verifying email and showing signup with google:

Screen.Recording.2026-01-23.at.8.53.51.PM.mp4

Trying to signup without verifying email:

Screen.Recording.2026-01-23.at.9.01.21.PM.mp4

Signed-off-by: Rakshit Yadav <yadavrakshit60@gmail.com>
Signed-off-by: Rakshit Yadav <yadavrakshit60@gmail.com>
Copy link
Member

@KarinePistili KarinePistili left a comment

Choose a reason for hiding this comment

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

Tested the flow and it did not work for me. Email did not arrive. I tried "Resend" button and it showed an error:
image

Signed-off-by: Rakshit Yadav <yadavrakshit60@gmail.com>
@rakshityadav1868 rakshityadav1868 marked this pull request as draft January 24, 2026 14:39
@rakshityadav1868 rakshityadav1868 marked this pull request as ready for review January 24, 2026 17:59
Signed-off-by: Rakshit Yadav <yadavrakshit60@gmail.com>
@sonarqubecloud
Copy link

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.

[Security Improvement]: Emails are not being verified during sign up.

2 participants