Skip to content

Conversation

@mandarini
Copy link
Contributor

Moved from supabase/auth-js#1011
Author: @j4w8n

What kind of change does this PR introduce?

Minor type fix.

What is the current behavior?

If a user has multiple providers associated with them, then Supabase adds a providers string array to their user.app_metadata, but this is missing from the UserAppMetadata type. I just happened to notice this while trying to work around an issue in a demo app, and there was no providers intellisense in my IDE.

image

What is the new behavior?

A providers?: string[] type is added to the interface.

Additional context

An example of multiple providers being associated to a user is if the user signs up with email/password, but then is allowed to add a phone number as well.

@coveralls
Copy link

coveralls commented Oct 9, 2025

Coverage Status

coverage: 95.276% (+12.8%) from 82.5%
when pulling 744ce43 on fix/add-app-metadata-type
into 188fa17 on master.

@mandarini mandarini added the auth-js Related to the auth-js library. label Oct 13, 2025
@mandarini mandarini added do-not-merge Do not merge this PR. and removed do-not-merge Do not merge this PR. labels Nov 6, 2025
issuedat
issuedat previously approved these changes Nov 7, 2025
Copy link
Contributor

@issuedat issuedat left a comment

Choose a reason for hiding this comment

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

LGTM ✅ Just added a suggestion for some typedocs to help understand the difference between the two types!

Thanks for the contribution 🙇

Co-authored-by: issuedat <165281975+issuedat@users.noreply.github.com>
@mandarini mandarini merged commit 2d862a1 into master Nov 7, 2025
25 checks passed
@mandarini mandarini deleted the fix/add-app-metadata-type branch November 7, 2025 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auth-js Related to the auth-js library.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants