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.

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

Add sending verification email after linking email address #1656

Merged
merged 3 commits into from
Jun 11, 2024

Conversation

nilsreichardt
Copy link
Member

In the future, we should only send emails to verified email addresses. Otherwise, it's likely that we'll have a high bounce rate, which could cause our emails to be flagged as spam.

Users with Google and Apple providers don't need to verify their email address.

Copy link

github-actions bot commented Jun 6, 2024

Visit the preview URL for this PR (updated for commit 63dd0f5):

https://sharezone-console-dev--pr1656-add-sending-verify-e-yf081nzz.web.app

(expires Fri, 14 Jun 2024 15:08:52 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 471536afe3f6ec4895d9ea75513730b515d17eb6

Copy link

github-actions bot commented Jun 6, 2024

Visit the preview URL for this PR (updated for commit 63dd0f5):

https://sharezone-website-dev--pr1656-add-sending-verify-e-aj8slo18.web.app

(expires Fri, 14 Jun 2024 15:08:51 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 372b0431a96247f908d9a97d5d865de1c8b3b04e

Copy link

github-actions bot commented Jun 6, 2024

Visit the preview URL for this PR (updated for commit 63dd0f5):

https://sharezone-test--pr1656-add-sending-verify-e-dqtdsl0j.web.app

(expires Fri, 14 Jun 2024 15:10:05 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4cb3ae61e1e018abfd9841fd3239f5b49ccc034b

@nilsreichardt nilsreichardt merged commit 29d9552 into main Jun 11, 2024
36 of 37 checks passed
@nilsreichardt nilsreichardt deleted the add-sending-verify-email-address branch June 11, 2024 16:32
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