mapzen / ios

Where you can find everything iOS from Mapzen
https://mapzen.com/projects/mobile/
Apache License 2.0
40 stars 24 forks source link

Fix pelias results archiving functionality #388

Open msmollin opened 6 years ago

msmollin commented 6 years ago

Pull in latest pelias release?

RaimundWege commented 6 years ago

We are currently testing different MapViews with Xcode 9 and can't use this SDK because of an issue in the pelias dependency:

pelias

Is related to #341

msmollin commented 6 years ago

Interesting - yeah we're aware of that issue but it's not causing compiling errors for us in our Sample App. Can you possible share some implementation code?

We fixed it as part of the Swift 4 migration for Pelias however we then ran into the linked cocoapod issue in #341, so we've been holding off on upgrading Pelias in the main SDK until that cocoapod issue got resolved and the subsequent PRs merged.

If it's causing you issues we can revert the Swift 4 migration and patch it in Swift 3.x.