search

imperiumlabs / GeoFirestore-iOS

GeoFirestore for iOS - Realtime location queries with Firestore
MIT License
127 stars 46 forks source link

Updated all dependencies including Firebase #29

Open patpatchpatrick opened 4 years ago

patpatchpatrick commented 4 years ago

I updated all of the pod dependencies, including Firebase, so that the most recent versions of Firebase/Firestore can be used. I tested everything in my project and it seems to be working fine.

AlexanderMarchant commented 4 years ago

Please be aware:

This update introduces a dependency on the 'GeoPoint' class to be mapped onto your swift object. If you do not use the GeoPoint mapping on your object, other features - such as locating documents within a radius - will not work correctly.

Previous possible mapping: var l: [Double]

New required mapping: var l: GeoPoint

Not a biggie, but GeoPoint does not conform to codable, therefore, I needed to create my own codable implementation and play around with the JSON before attempting to decode my object containing the GeoPoint.

My workaround being I removed the GeoPoint JSON from the JSON object and manually instantiated a GeoPoint object then assigned it to the objects field after decoding the other JSON.

Otherwise, thanks for this pull request.