Skip to content

Signature changes to match approved API (auth-exp) #3990

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

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions packages-exp/auth-types-exp/index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -982,12 +982,12 @@ export abstract class MultiFactorResolver {
* The list of hints for the second factors needed to complete the sign-in for the current
* session.
*/
hints: MultiFactorInfo[];
readonly hints: MultiFactorInfo[];
/**
* The session identifier for the current sign-in flow, which can be used to complete the second
* factor sign-in.
*/
session: MultiFactorSession;
readonly session: MultiFactorSession;
/**
* A helper function to help users complete sign in with a second factor using an
* {@link MultiFactorAssertion} confirming the user successfully completed the second factor
Expand Down Expand Up @@ -1314,7 +1314,7 @@ export interface UserCredential {
/**
* The provider which was used to authenticate the user.
*/
providerId: ProviderId | null;
providerId: string | null;
/**
* The type of operation which was used to authenticate the user (such as sign-in or link).
*/
Expand Down Expand Up @@ -1383,7 +1383,7 @@ export interface AdditionalUserInfo {
/**
* Identifier for the provider used to authenticate this user.
*/
readonly providerId: ProviderId | null;
readonly providerId: string | null;
/**
* The username if the provider is GitHub or Twitter.
*/
Expand Down