Skip to content
This repository has been archived by the owner on Jun 8, 2023. It is now read-only.

fix: fix verification mail #316

Merged
merged 2 commits into from
Nov 28, 2022
Merged

fix: fix verification mail #316

merged 2 commits into from
Nov 28, 2022

Conversation

asbiin
Copy link
Member

@asbiin asbiin commented Nov 27, 2022

This fixes #273

@djaiss
Copy link
Member

djaiss commented Nov 27, 2022

So basically if there is no STMP set up, we mark the email as verified. This should indeed solve all our issues.

@asbiin
Copy link
Member Author

asbiin commented Nov 28, 2022

That's it. Or you can use another mail driver, then you'll have to setup it accordingly.

@sonarcloud
Copy link

sonarcloud bot commented Nov 28, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

80.0% 80.0% Coverage
0.0% 0.0% Duplication

@asbiin asbiin merged commit 8c35ddd into main Nov 28, 2022
@asbiin asbiin deleted the 20221127-fix-verification-mail branch November 28, 2022 10:03
asbiin added a commit to monicahq/monica that referenced this pull request Mar 31, 2023
asbiin added a commit to monicahq/monica that referenced this pull request May 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker Setup using log mailer (some gotchas)
2 participants