-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Breaking changes to signature of snapshots and setData in cloud_firestore #536
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
## 0.7.0 | ||
|
||
* **Breaking change**. snapshots is now a method instead of a getter. | ||
* **Breaking change**. setData uses optional arguments instead of SetOptions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe use code font on snapshots
, setData
, and SetOptions
to avoid having the entry look like a spelling error with lowercase word following the full stop.
[nit] optional arguments => named argument?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
/// document does not yet exist, it will be created. | ||
/// | ||
/// If [merge] is true, the provided data will be merged into an | ||
/// existing document. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please document the default semantics: overwrite?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@@ -61,10 +61,10 @@ class WriteBatch { | |||
/// | |||
/// If the document does not yet exist, it will be created. | |||
/// | |||
/// If you pass [SetOptions], the provided data will be merged into an | |||
/// If [merge] is true, the provided data will be merged into an |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
…tore (flutter#536) These changes are required to support the metadata argument to snapshots and other future API changes.
…tore (flutter#536) These changes are required to support the metadata argument to snapshots and other future API changes.
No description provided.