Skip to content

Conversation

issuedat
Copy link
Contributor

@issuedat issuedat commented Oct 1, 2025

This PR adds support for sending the user an email notification when their phone number has been changed.

3 new environment variables are introduced:

  • GOTRUE_MAILER_SUBJECTS_PHONE_CHANGED_NOTIFICATION: Email subject to use for phone changed notification.
  • GOTRUE_MAILER_TEMPLATES_PHONE_CHANGED_NOTIFICATION: The URL to specify a custom template.
  • GOTRUE_MAILER_NOTIFICATIONS_PHONE_CHANGED_ENABLED: whether the notification is enabled or not.

The feature is disabled by default. To enable it, the GOTRUE_MAILER_NOTIFICATIONS_PHONE_CHANGED_ENABLED environment variable must be set to true.

@issuedat issuedat requested a review from a team as a code owner October 1, 2025 11:55
@coveralls
Copy link

coveralls commented Oct 1, 2025

Pull Request Test Coverage Report for Build 18161668709

Details

  • 29 of 47 (61.7%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.02%) to 67.598%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/api/verify.go 10 13 76.92%
internal/api/mail.go 9 15 60.0%
internal/mailer/templatemailer/templatemailer.go 0 9 0.0%
Totals Coverage Status
Change from base Build 18138525531: -0.02%
Covered Lines: 13381
Relevant Lines: 19795

💛 - Coveralls

Copy link
Contributor

@cstockton cstockton left a comment

Choose a reason for hiding this comment

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

🚀

@issuedat issuedat merged commit 21f3070 into master Oct 1, 2025
6 of 7 checks passed
@issuedat issuedat deleted the iat/auth-840-phone-number-changed-notification branch October 1, 2025 17:28
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.

3 participants