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

feat: email verification #145

Merged
merged 12 commits into from
Dec 16, 2024
Merged

feat: email verification #145

merged 12 commits into from
Dec 16, 2024

Conversation

christophenne
Copy link
Member

@christophenne christophenne commented Dec 16, 2024 •
edited by pmig
Loading

⚠️ needs DB migration

ALTER TABLE UserAccount ADD COLUMN email_verified_at TIMESTAMP;

To update all existing users to be verified:

UPDATE UserAccount SET email_verified_at = current_timestamp WHERE email_verified_at IS NULL;

@christophenne
WIP
b922183 
Signed-off-by: christophenne <christoph.enne@glasskube.eu>
@christophenne
Merge branch 'main' into christoph/email-verification
b9cbb35 
# Conflicts:
#	internal/handlers/auth.go
#	internal/mailtemplates/templates.go
@christophenne
feat: email verification
5974570 
Signed-off-by: christophenne <christoph.enne@glasskube.eu>
@christophenne
Merge branch 'main' into christoph/email-verification
f0dcd3d
@christophenne
feat: email verify
e265718 
Signed-off-by: christophenne <christoph.enne@glasskube.eu>
@christophenne
feat: email verify
ee1cc9c 
Signed-off-by: christophenne <christoph.enne@glasskube.eu>
@christophenne christophenne changed the title feat: email verification [WIP] feat: email verification Dec 16, 2024
@christophenne christophenne requested a review from kosmoz December 16, 2024 10:19
@christophenne
chore: remove TODO
39499d5 
Signed-off-by: christophenne <christoph.enne@glasskube.eu>
kosmoz
kosmoz reviewed Dec 16, 2024
View reviewed changes
internal/handlers/auth.go Outdated Show resolved Hide resolved
internal/handlers/settings.go Outdated Show resolved Hide resolved
@christophenne
fix: only allow update when already verified and wrong status code
12d79db 
Signed-off-by: christophenne <christoph.enne@glasskube.eu>
@christophenne
chore: lint
7be35b6 
Signed-off-by: christophenne <christoph.enne@glasskube.eu>
@christophenne christophenne requested a review from kosmoz December 16, 2024 11:23
@christophenne
Merge branch 'main' into christoph/email-verification
e4c02d8
@christophenne
chore: query escape token
fa47dab 
Signed-off-by: christophenne <christoph.enne@glasskube.eu>
@christophenne
chore: lint
562b9f2 
Signed-off-by: christophenne <christoph.enne@glasskube.eu>
@christophenne christophenne mentioned this pull request Dec 16, 2024
Merged
@kosmoz kosmoz merged commit e78be22 into main Dec 16, 2024
2 checks passed
@kosmoz kosmoz deleted the christoph/email-verification branch December 16, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kosmoz kosmoz kosmoz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@christophenne @kosmoz