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.

Sign up for GitHub

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

Jump to bottom

Mail verification #54

Closed
PrabuckiDominik opened this issue Aug 29, 2024 · 0 comments · Fixed by #57
Closed

Mail verification #54

PrabuckiDominik opened this issue Aug 29, 2024 · 0 comments · Fixed by #57
Assignees
@PrabuckiDominik

Comments

@PrabuckiDominik
Copy link
Collaborator

After registration, verification mail is sent to the user in his mailbox, but he is able to log in without confirming his email address.

Let's change that - if a user wants to log in to his account (if his mail is not verified), he will be redirected to the page to resend verification mail, and a reminder that in order to login he needs to verify it.
@PrabuckiDominik PrabuckiDominik self-assigned this Aug 29, 2024
@PrabuckiDominik PrabuckiDominik mentioned this issue Aug 29, 2024
Merged
@PrabuckiDominik PrabuckiDominik linked a pull request Aug 29, 2024 that will close this issue
#54 - mail verification #57
Merged
PrabuckiDominik added a commit that referenced this issue Aug 30, 2024
@PrabuckiDominik @kamilpiech97
#54 - mail verification (#57)
9986a09 
* Add verified middleware

* Add resend verification mail and must verify mail to access user dashboard

* Fix code style

* Add tests for unverifiedUser

* Apply suggestions from code review

Co-authored-by: Kamil Piech <[email protected]>

* Fix misspelled word

* Add redirect to home page if user tries to verify email via mail and is not logged in.
Add automatic log in after registration

---------

Co-authored-by: Kamil Piech <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PrabuckiDominik PrabuckiDominik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#54 - mail verification blumilksoftware/interns2024b
1 participant
@PrabuckiDominik