Skip to content

Conversation

rsgowman
Copy link
Member

Previously, only the last mutation would actually be visible. (Mutations would not actually be discarded, but you wouldn't be able to actually see them until the client came online again.)

@rsgowman rsgowman force-pushed the rsgowman/mult_offline_mutations branch from 5be0b75 to e76c78c Compare January 11, 2018 01:13
@@ -54,6 +54,8 @@ NS_ASSUME_NONNULL_BEGIN
- (void)shutdownWithCompletion:(nullable void (^)(NSError *_Nullable error))completion
NS_SWIFT_NAME(shutdown(completion:));

- (instancetype)firestoreWithConfiguredClient;
Copy link
Contributor

Choose a reason for hiding this comment

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

This shouldn't be required.

@wilhuff wilhuff merged commit 1bafdb4 into master Jan 11, 2018
@wilhuff wilhuff deleted the rsgowman/mult_offline_mutations branch January 11, 2018 16:50
@paulb777 paulb777 mentioned this pull request Jan 11, 2018
bstpierr added a commit that referenced this pull request Jan 12, 2018
* master:
  Remove over-aggressive closeWithFinalState: delegate assert. (#656)
  Revert "Fully qualify protoc-generated outputs (#626)" (#653)
  Updates Auth ChangeLog (#645)
  Update CHANGELOG for Firestore v0.10.0 (#649)
  Fix FSTLocalDocumentsView to allow multiple mutations while offline (#644)
minafarid pushed a commit to minafarid/firebase-ios-sdk that referenced this pull request Jun 6, 2018
…irebase#644)

* Fix FSTLocalDocumentsView to allow multiple mutations while offline.

Previously, only the last mutation would actually be visible.
@firebase firebase locked and limited conversation to collaborators Nov 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants