Conversation
Vertex AI Mock Responses Check
|
|
Size Report 1Affected Products
Test Logs |
Size Analysis Report 1Affected Products
Test Logs |
| // @public | ||
| export function writeBatch(firestore: Firestore): WriteBatch; | ||
|
|
||
|
|
There was a problem hiding this comment.
i did not see this API proposal 😮
There was a problem hiding this comment.
It was a blank document. :)
MarkDuckworth
left a comment
There was a problem hiding this comment.
LGTM. However, the description says this fixes #6824, but that is related to Auth. Not a big deal because you're apply this to auth at the same time, but in a different PR.
Righto, removed that Fixes comment in the PR. Thanks for the review! |
Discussion
Update
connectFirestoreEmulatorto support its invocation more than once. If the Firestore instance is already in use, andconnectFirestoreEmulatoris invoked with the same configuration, then the invocation will now succeed instead of assert.The implementation takes the Data Connect implementation as inspiration. Data Connect stores the parameters passed to
connectDataConnectEmulatoron the instance of Data Connect itself, so that they can be quickly checked to see if subsequent invocations match. This PR implements a similar storage and compare process with the optionalemulatorOptionsparameter (host and port are already stored).This PR unlocks support for SSR frameworks which render the page numerous times with the same instances of Firestore. Before this PR customers were required to guard against calling
connectFirestoreEmulatorin their SSR logic, which added to code complexity. Now the Firebase SDK does that guarding logic so that our users' apps don't have to.Testing
CI, new tests added to
packages/firestore/test/integration/api/validation.test.tsandpackages/firestore/test/unit/api/database.test.ts.API Changes
N/A