Skip to content
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

Fix backfill for subcollections #51

Closed
wants to merge 1 commit into from

Conversation

guenth39
Copy link

@guenth39 guenth39 commented Apr 20, 2023

Change Summary

With these changes, the backfill will now work also for subcollections, not only for "root" collection. If the collection path is something like users/{userId}/items the sync has already worked, but the backfill did not as the documents were queried with this collection path, but that did not work. Instead, we use the collectionGroup feature now with only the last part of the path.

Closes #17

PR Checklist

@guenth39
Copy link
Author

Closed in favor of #52 to prevent mixing up different changes.

@guenth39 guenth39 closed this Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support backfill documents in subcollections
1 participant