Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Replace interactive map view with step table in example application #523

Closed
MaximAlien opened this issue Mar 1, 2021 · 4 comments
Closed

Replace interactive map view with step table in example application #523

MaximAlien opened this issue Mar 1, 2021 · 4 comments
Assignees
@S2Ler
Labels
build op-ex Refactoring, Tech Debt or any other operational excellence work. UI
Milestone
v2.0

Comments

@MaximAlien
Copy link
Contributor

Mapbox Directions example application should be updated to Mapbox Maps SDK v10.
@1ec5
Copy link
Contributor

1ec5 commented Mar 3, 2021

Map SDK v10 introduces a more onerous installation process. The example application doesn’t really make use of the map SDK, so I’m thinking it would be less of a hassle and more maintainable to migrate the example application to MapboxStatic.swift instead.

@1ec5 1ec5 added the op-ex Refactoring, Tech Debt or any other operational excellence work. label Mar 3, 2021
@truburt truburt added this to the v2.0.0 (RC) milestone Mar 5, 2021
@1ec5 1ec5 modified the milestones: v2.0.0 (RC), v2.0.0 (GA) Mar 5, 2021
@truburt truburt added question and removed question labels Apr 2, 2021
@1ec5
Copy link
Contributor

1ec5 commented Apr 26, 2021

The example application doesn’t really make use of the map SDK, so I’m thinking it would be less of a hassle and more maintainable to migrate the example application to MapboxStatic.swift instead.

Or even remove the map entirely in favor of a table view that can present the information that’s currently relegated to standard output.

@1ec5 1ec5 changed the title Update Mapbox Directions example application to Mapbox Maps SDK v10. Replace interactive map view with step table in example application May 4, 2021
@S2Ler S2Ler self-assigned this May 28, 2021
@S2Ler S2Ler mentioned this issue Jun 1, 2021
Merged
@S2Ler S2Ler added the PR Ready label Jun 1, 2021
@S2Ler
Copy link
Contributor

S2Ler commented Jun 4, 2021

Fixed in #548

@S2Ler S2Ler closed this as completed Jun 4, 2021
@S2Ler
Copy link
Contributor

S2Ler commented Jun 4, 2021

Further improvements tracked in #549

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

@S2Ler S2Ler

Labels
build op-ex Refactoring, Tech Debt or any other operational excellence work. UI
Projects
None yet
Milestone
v2.0
Development

No branches or pull requests
4 participants
@S2Ler @1ec5 @MaximAlien @truburt