Skip to content

Conversation

issuedat
Copy link
Contributor

Introduces EmailOptional configuration to allow sign-ins/ups when the provider does not return an email address.

Related: supabase/supabase#38788

@coveralls
Copy link

Pull Request Test Coverage Report for Build 18130622525

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 5 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.02%) to 54.784%

Files with Coverage Reduction New Missed Lines %
internal/gen/keys/keys.go 5 12.9%
Totals Coverage Status
Change from base Build 18130359431: -0.02%
Covered Lines: 6390
Relevant Lines: 11664

💛 - Coveralls

@issuedat issuedat marked this pull request as ready for review October 3, 2025 09:53
@issuedat issuedat requested a review from a team as a code owner October 3, 2025 09:53
Copy link
Contributor

@sweatybridge sweatybridge left a comment

Choose a reason for hiding this comment

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

Looks good. Just need to add the new field to default config.toml as documentation https://github.com/supabase/cli/blob/develop/pkg/config/templates/config.toml#L266

@issuedat
Copy link
Contributor Author

issuedat commented Oct 3, 2025

Thanks @sweatybridge! Just added the config to the template ✅

@sweatybridge sweatybridge merged commit 4d02964 into develop Oct 3, 2025
15 checks passed
@sweatybridge sweatybridge deleted the iat/email-optional branch October 3, 2025 14:48
@github-actions github-actions bot mentioned this pull request Oct 7, 2025
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