Skip to content

added AuthTokenResult #142

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 5 commits into from
Feb 3, 2021
Merged

added AuthTokenResult #142

merged 5 commits into from
Feb 3, 2021

Conversation

suntrix
Copy link
Contributor

@suntrix suntrix commented Jan 28, 2021

Fixes #107

Comment on lines +49 to +50
val signInProvider: String?
val token: String?
Copy link
Member

Choose a reason for hiding this comment

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

can they be null?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately the properties in Android SDK are marked as @Nullable, so didn't want to risk a crash.

@nbransby nbransby merged commit 40ba027 into GitLiveApp:master Feb 3, 2021
@suntrix suntrix deleted the auth-api-coverage branch February 6, 2021 13:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add FirebaseUser.idToken with claims to firebase-auth
2 participants