Skip to content
This repository was archived by the owner on Mar 4, 2026. It is now read-only.

fix: Handles identical document ids from different collections.#1599

Merged
schmidt-sebastian merged 2 commits into
masterfrom
wuandy/IdentityDocIdForBundles
Aug 26, 2021
Merged

fix: Handles identical document ids from different collections.#1599
schmidt-sebastian merged 2 commits into
masterfrom
wuandy/IdentityDocIdForBundles

Conversation

@wu-hui

@wu-hui wu-hui commented Aug 26, 2021

Copy link
Copy Markdown
Contributor

Fixes #1598

@wu-hui wu-hui requested review from a team August 26, 2021 14:56
@product-auto-label product-auto-label Bot added the api: firestore Issues related to the googleapis/nodejs-firestore API. label Aug 26, 2021
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 26, 2021

@schmidt-sebastian schmidt-sebastian left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the quick fix!

@schmidt-sebastian schmidt-sebastian merged commit 745b608 into master Aug 26, 2021
@schmidt-sebastian schmidt-sebastian deleted the wuandy/IdentityDocIdForBundles branch August 26, 2021 16:17
@willcalderbank

Copy link
Copy Markdown

Thanks for the quick fix! Great work!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: firestore Issues related to the googleapis/nodejs-firestore API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bundles use id as unique identifier however id unique-ness is not enforced across all collections

4 participants