This repository was archived by the owner on Feb 22, 2023. It is now read-only.
Support for update and merge writes#298
Merged
collinjackson merged 6 commits intoflutter:masterfrom Dec 5, 2017
Merged
Conversation
collinjackson
commented
Dec 5, 2017
| /// By providing the [SetOptions] objects returned by [merge], the [setData()] | ||
| /// calls on [DocumentReference] can be configured to perform granular merges | ||
| /// instead of overwriting the target documents in their entirety. | ||
| class SetOptions { |
Contributor
Author
There was a problem hiding this comment.
The Android Firestore SDKs provides more advanced SetOptions functionality, but it doesn't seem to be available yet on iOS and I only implemented the lowest common denominator. I've added a comment to this effect below.
To make the API more Firebase-y and for future compatibility I'm making this a class instead of more Flutter-y optional named arguments.
19d7d7c to
87dbeb9
Compare
Hixie
reviewed
Dec 5, 2017
| const <String, dynamic>{'merge': true}, | ||
| ); | ||
|
|
||
| const SetOptions._(this._data); |
Contributor
There was a problem hiding this comment.
i almost missed this constructor. this is why we have the style rule that says constructors go first. :-)
Contributor
|
|
|
Which library or package do we need to import in order to access the |
julianscheel
pushed a commit
to jusst-engineering/plugins
that referenced
this pull request
Mar 11, 2020
julianscheel
pushed a commit
to jusst-engineering/plugins
that referenced
this pull request
Mar 11, 2020
Fix documentation list format.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is required for the todo list app: https://github.com/flutter/udacity-course/pull/26