aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Reimplement C++ logging (#1313)Gravatar Gil2018-05-23
| | | | | | | * Add direct support for formatting Objective-C objects * Rewrite log.h * Convert FSTWarn to LOG_WARN * Convert FSTLog to LOG_DEBUG * Remove FSTLogger
* Directly validate collection and document paths (#1316)Gravatar Gil2018-05-22
| | | | | | ... instead of relying on HARD_ASSERTS in resource_path.cc to catch them. This fixes an integration test broken in #1304.
* Remove extraneous braces (#1315)Gravatar Gil2018-05-22
|
* Add a HARD_ASSERT C++ assertion macro (#1304)Gravatar Gil2018-05-22
| | | | | | * Add HARD_ASSERT * Use HARD_ASSERT * Remove FIREBASE_ASSERT * Remove StringPrintf
* Add namespace alias for v1beta1 (#1310)Gravatar rsgowman2018-05-22
| | | Roughly s/google::firebase::v1beta1/v1beta1/g
* Firestore project fixes (#1301)Gravatar Gil2018-05-22
| | | | | | * Move GoogleTest.podspec out of the Tests tree * Rename GoogleTests to CoreTests * Move CoreTests to the top-level * Move FSTGoogleTestTests.mm into core/test
* [De]serialize non-empty Document instances (#1284)Gravatar rsgowman2018-05-22
| | | | | | | * [De]serialize non-empty Document instances Still TODO: - NoDocument - ErrorHandling
* add macro to remove deprecated warning (#1295)Gravatar Chen Liang2018-05-22
|
* Move library registration to +initialize. (#1305)Gravatar Ryan Wilson2018-05-22
|
* fix build for Firestore copybara import (#1308)Gravatar zxu2018-05-22
| | | | | | * fix "target of using declaration conflicts with declaration already in scope" * fix "call to implicitly-deleted copy constructor of *unique_ptr*"
* Add conformance to data collection switch for Analytics. (#1296)Gravatar Ryan Wilson2018-05-21
| | | | | | * Add conformance to data collection switch for Analytics. * Fix method documentation per PR comments.
* Add a C++ native StringFormat (#1289)Gravatar Gil2018-05-21
| | | | * Add StringFormat * Use StringFormat
* ignore deprecated warning of get token method (#1292)Gravatar Chen Liang2018-05-18
|
* fix lint to run copybara (#1290)Gravatar zxu2018-05-17
| | | | | | * fix lint to run copybara * address changes
* b/79432277: Limit Queries to only a single array-contains clause. (#1286)Gravatar Michael Lehenbauer2018-05-16
|
* Use brackets to import system header sqlite3.h (#1285)Gravatar Paul Beusterien2018-05-16
|
* Refactoring serializer.cc (#1281)Gravatar rsgowman2018-05-16
| | | | Mostly to make existing methods a bit more general to support followup PR (which will allow encoding/decoding documents with contents.)
* Merge pull request #1282 from firebase/pb-merge501Gravatar Paul Beusterien2018-05-15
|\ | | | | Merge release-5.0.1 to master
| * mergeGravatar Paul Beusterien2018-05-15
|/|
* | [De]serialize empty Document instances (#1262)Gravatar rsgowman2018-05-15
| | | | | | | | | | | | | | | | | | * [De]serialize empty Document instances Still TODO: - non-empty - NoDocument - ErrorHandling
* | More macOS/static library prep (#1280)Gravatar Gil2018-05-15
| | | | | | | | | | | | | | | | | | | | * Avoid direct use of generated umbrella headers These: * don't exist in static library builds * have different names in multi-platform builds * Only iOS targets depend upon Firebase/Core
* | Test Core's build version matches FIROption version (#1279)Gravatar Paul Beusterien2018-05-15
| |
* | Prep for direct macOS support in Firestore pod (#1269)Gravatar Gil2018-05-14
| | | | | | | | | | | | | | | | | | * Add Firestore/Example/App group * Move "Example For Firestore" into App * Rename "Example For Firestore" to "iOS" * Update plist file location
* | Explicitly import FIRDataSnapshot in private header. (#1276)Gravatar Ryan Wilson2018-05-14
| |
* | Temporarily disable Messaging pod lints (#1277)Gravatar Ryan Wilson2018-05-14
| | | | | | | | | | There's an unreleased change in Core that Messaging relies on, but Travis currently pulls in the publicly released version of Core causing the linting to fail. This should be re-enabled once either a) Core is released, or b) we support pointing to local pods for dependencies on Travis - whichever of the two come first. An issue will be opened to re-enable the linting.
* | Add global data collection switch. (#1219)Gravatar Ryan Wilson2018-05-14
| | | | | | | | | | | | | | * Addition of global data collection switch. * Added Messaging conformance to data switch. Also formatted code. * Move data collection flag internal until all SDKs conform to it. * Formatting in response to code review.
* | Firestore: add a changelog entry on supporting concurrent queues. (#1274)Gravatar Konstantin Varlamov2018-05-14
| |
| * Merge pull request #1273 from firebase/pb-cp-mlkit-firloggerGravatar Paul Beusterien2018-05-14
| |\ | | | | | | Add MLKit to FIRLogger constants. (#1263)
| | * Add MLKit to FIRLogger constants. (#1263)Gravatar Ryan Wilson2018-05-14
| |/
| * Merge pull request #1271 from firebase/pb-cp-all_load-fixGravatar Paul Beusterien2018-05-14
| |\ | | | | | | Eliminate duplicate symbol in CocoaPods -all_load build (#1223)
| * | Version updates for 5.0.1 (#1272)Gravatar Paul Beusterien2018-05-14
| | |
* | | Trivial/noop fix to the serializer (#1264)Gravatar rsgowman2018-05-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Was using google_firestore_v1beta1_MapValue_FieldsEntry_key_tag to tag the key/value pair. (But that tag should be used for the *key* of the key/value pair, not the pair itself.) Switched to using google_firestore_v1beta1_MapValue_fields_tag. This previously worked anyways by coincidence. These two values happen to be the same. (But it caused me all sorts of confusion as I adapted this for Document contents.)
| | * Eliminate duplicate symbol in CocoaPods -all_load build (#1223)Gravatar Paul Beusterien2018-05-14
| |/
* | Fix cron build - FirebaseMessaging static library has warnings (#1265)Gravatar Paul Beusterien2018-05-13
| |
* | Clean up implicit retain self warnings introduced recently (#1268)Gravatar Gil2018-05-13
| |
* | Update travis to use CocoaPods 1.5.2 (#1103)Gravatar Paul Beusterien2018-05-13
| | | | | | | | | | | | | | | | * Update travis to use CocoaPods 1.5.2 * CODE_SIGNING_ALLOWED=YES * Remove workaround and iPhone 8 * Remove #1210 static library Podfile workaround * Add -all_load to Firestore Example so all C++ is available for tests
* | Firestore C++: make FSTDispatchQueue delegate to C++ implementation (#1240)Gravatar Konstantin Varlamov2018-05-11
| | | | | | | | | | FSTDispatchQueue now doesn't contain any logic of its own and instead just passes through all method calls to AsyncQueue (backed by an ExecutorLibdispatch).
* | Add static library pod linting (#1259)Gravatar Paul Beusterien2018-05-11
| | | | | | | | * add if_cron script
* | Add MLKit to FIRLogger constants. (#1263)Gravatar Ryan Wilson2018-05-11
| |
* | Refactor serializer (#1250)Gravatar rsgowman2018-05-11
| | | | | | | | | | Moved Tag, Reader, Writer from serializer.cc's anon namespace to firebase::firestore::nanopb This should be bug-for-bug compatible. No changes were made to the moved methods.
* | Firestore: use C++ Executor for user queue. (#1238)Gravatar Konstantin Varlamov2018-05-10
| | | | | | | | | | | | | | FSTDispatchQueue enforces serial execution, which is inappropriate for user queue, because a user may configure usage of a concurrent queue in settings, breaking FSTDispatchQueue invariants. Instead, use C++ ExecutorLibdispatch directly.
* | Firestore C++: quick fix for flaky tests in AsyncQueue and Executor (#1245)Gravatar Konstantin Varlamov2018-05-10
| |
* | Firestore C++: add missing tests for Executor (#1247)Gravatar Konstantin Varlamov2018-05-10
| |
* | Add remaining tests to Xcode (#1256)Gravatar Gil2018-05-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add classes in testutil and util to the project * Exclude util/iterator_adaptors_test which requires gmock * Exclude remote/serializer_test which is missing some paths * Add iterator_adaptors_test and add gmock support * Add gmock support to the GoogleTest podspec we vendor * Add iterator_adaptors_test.cc to the Xcode project * Add a script that verifies all tests are referenced in the project * Add a travis check that all tests are referenced in the project * Review feedback * Moar feedback
* | Disable Firestore pod lint subspec checking (#1258)Gravatar Paul Beusterien2018-05-10
| |
* | Add missing tests to the Firestore project (#1254)Gravatar Gil2018-05-09
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix order of project elements * Add immutable container tests to the Firestore project * Fix style problems in TransformOperations * Method names should have an initial upper case character (except getters/setters) * Pointers bind to the type not the name in C++ * Fix unsigned/signed comparison warning * Make transform_operations.h Objective-C++ only It turns out that you can't conditionally define virtual methods. This causes multiple versions of the vtable to be emitted and when the linker resolves all the vtable definitions for a class it picks one arbitrarily causing failures for callers that expect some of the conditional methods to be present. * Add precondition_test and field_mask_test to the project * Add field_transform_test and transform_operations_test to the project * Clean up missing newline/excess comments * Use braced initialization to avoid linter false positive on std::transform
* | Add back pod spec lint checking (#1252)Gravatar Paul Beusterien2018-05-09
| | | | | | | | | | | | | | | | | | * Add back pod spec lint checking * Separate pod lib lint to stages * order unit tests before lints * add lint to install_prereqs and remove default * consolidate pod_install and install_prereqs scripts * workaround travis 10 minute timeout for Firestore lint
* | Remove final vestiges of FSTSnapshotVersion (#1253)Gravatar Gil2018-05-09
| |
* | Align folders with actual test locations in the Firestore project (#1251)Gravatar Gil2018-05-09
| |
* | [De]serialize Timestamps via the remote Serializer (#1233)Gravatar rsgowman2018-05-09
| |