Skip to content

fix setup && bugs #202

Open
chizhavko wants to merge 9 commits into
mainfrom
max-fix-project-setup
Open

fix setup && bugs #202
chizhavko wants to merge 9 commits into
mainfrom
max-fix-project-setup

Conversation

@chizhavko
Copy link
Copy Markdown

-Move project setup from cocoapods to spm
-Fix crashes
-Updated code base and improved examples with new SDK features
-Update readme

Comment thread readme.md
@@ -10,16 +10,14 @@ A collection of examples showing off the [Mapbox Navigation SDK](https://github.

_Installation with CocoaPods_
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's change to SPM now

Comment thread readme.md
1. `git clone https://github.com/mapbox/mapbox-navigation-ios-examples.git`
1. `cd mapbox-navigation-ios-examples`
1. Open project and wait for couple of minutes. SPM need to load mapbox Navigation SDK dependencies. ☕️
1. Go to your [Mapbox account dashboard](https://account.mapbox.com/) and create an access token that has the `DOWNLOADS:READ` scope. **PLEASE NOTE: This is not the same as your production Mapbox API token. Make sure to keep it private and do not insert it into any Info.plist file.** Create a file named `.netrc` in your home directory if it doesn’t already exist, then add the following lines to the end of the file:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This step should be done before a user tries to resolve packages, otherwise, SPM won't be able to download binary artifacts.

lineLayer.lineCap = .constant(.round)

return lineLayer
func changeMapStyle(_ navigationMapView: NavigationMapView?) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Intention of this example is to showcase the ability to change route line styling via delegates. I'm not sure that we need this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants