-
Notifications
You must be signed in to change notification settings - Fork 948
Add getDoc() #3133
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
Add getDoc() #3133
Conversation
…irebase-js-sdk into mrschmidt/initializefirestore
This will be tested when getDoc() is added.
|
||
// Kick off initializing the datastore but don't actually wait for it. | ||
// eslint-disable-next-line @typescript-eslint/no-floating-promises | ||
PlatformSupport.getPlatform() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This code is inspired by similar code in the main Firestore class.
return this._initializationDone.promise; | ||
} | ||
|
||
private _makeDatabaseInfo(): DatabaseInfo { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
/* eslint-disable no-restricted-globals */ | ||
|
||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved this code as importing this entire class breaks the Lite compile with typings conflicts.
Binary Size ReportAffected SDKs
Test Logs
|
51bf5a2
to
9be9cf6
Compare
9be9cf6
to
58a84d1
Compare
…sdk into mrschmidt/snapshot
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.