Empty fields should not be saved to Realm/Atlas

WildAid / o-fish-android

Android app for the Officer's Fishery Information Sharing Hub (O-FISH). The mobile app allows fisheries officers to document and share critical information gathered during a routine vessel inspection.

Apache License 2.0

22 stars 24 forks source link

Empty fields should not be saved to Realm/Atlas #15

Closed o-fish-wildaid closed 4 years ago

o-fish-wildaid commented 4 years ago

If a field is empty, it should not be saved as "empty string" in Atlas, the document should be sparse. If it's possible, the same for Realm, to minimize data being sent back and forth.

As of Monday, 18 May 2020 we're not ready for this to be done. We first need to get iOS, Android, web and Atlas working with a common data set. After that, I will assess whether making simple fields optional works with Realm Sync - and then we will decide whether to proceed.

Sheeri commented 4 years ago

Waiting on Andrew to test if we're ready yet, he will test when he removes the ViewModel from iOS.

am-MongoDB commented 4 years ago

This isn't how Realm sync works. If the field exists in the mobile app's Objects then they will be in the Atlas document.

Issue https://github.com/WildAid/o-fish-android/issues/297 created to make lastDelivery optional