Skip to content

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

Merged
merged 44 commits into from
Jun 4, 2020
Merged

Add getDoc() #3133

merged 44 commits into from
Jun 4, 2020

Conversation

schmidt-sebastian
Copy link
Contributor

No description provided.


// Kick off initializing the datastore but don't actually wait for it.
// eslint-disable-next-line @typescript-eslint/no-floating-promises
PlatformSupport.getPlatform()
Copy link
Contributor Author

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 {
Copy link
Contributor Author

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 */

/**
Copy link
Contributor Author

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.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented May 28, 2020

Binary Size Report

Affected SDKs

  • @firebase/firestore/lite

    Type Base (3361a63) Head (54b780b) Diff
    main 111 kB 114 kB +2.92 kB (+2.6%)

Test Logs

@schmidt-sebastian schmidt-sebastian force-pushed the mrschmidt/newgetdocument branch from 51bf5a2 to 9be9cf6 Compare May 29, 2020 03:17
@schmidt-sebastian schmidt-sebastian force-pushed the mrschmidt/newgetdocument branch from 9be9cf6 to 58a84d1 Compare May 29, 2020 03:44
Copy link
Contributor

@rafikhan rafikhan left a comment

Choose a reason for hiding this comment

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

LGTM

@schmidt-sebastian schmidt-sebastian changed the base branch from mrschmidt/snapshot to master June 4, 2020 23:24
@schmidt-sebastian schmidt-sebastian merged commit 94f3c34 into master Jun 4, 2020
@firebase firebase locked and limited conversation to collaborators Jul 5, 2020
@schmidt-sebastian schmidt-sebastian deleted the mrschmidt/newgetdocument branch November 9, 2020 22:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants