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.

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

crash when route ends: [Storyboard] Unknown class _TtC40maplibre-navigation-ios_MapboxNavigation13RatingControl in Interface Builder file. #57

Closed
michaelkirk opened this issue Jun 13, 2024 · 2 comments

Comments

@michaelkirk
Copy link
Collaborator

michaelkirk commented Jun 13, 2024

Steps to Trigger Behavior

  1. Start navigation and wait for it to complete (e.g. with a simulatedLocationManger).

Link to Minimal Reproducible Example

I don't have an example handy, but if you're interested I can put one together.

Expected Behavior

The endOfRouteViewController should be presented

Actual Behavior

The app crashes due to:

Screenshot 2024-06-12 at 17 42 55

[Storyboard] Unknown class RatingControl in Interface Builder file.

Screenshots (if applicable)

Version(s) affected

  • Platform: iOS
  • OS version: iOS17
  • Device type: simulator (haven't tried on device)
  • Emulator/ Simulator: yes
  • Development OS: osx 14.5 (23F79)
  • maplibre-navigation-ios: HEAD eaee92b
  • MapLibre GL version [e.g. 6.3.0]: 6.4.1

Additional context

One thing that confused me is that there seems to be another bug whereby the end of route logic will never fire unless you are using spoken directions, which I hadn't been. I'll open up an issue for that momentarily. (update - here it is: #58)

michaelkirk added a commit to michaelkirk/maplibre-navigation-ios that referenced this issue Jun 13, 2024
Previously, having voice instructions was a requirement of ending the
route.

Now we'll end the route when banner instructions are done *and* only
consider voice instructions if there were any to begin with.

Note: This currently crashes do to another issue with the end of route
view controller: see maplibre#57
@michaelkirk
Copy link
Collaborator Author

I pushed a fix to #58.

@michaelkirk
Copy link
Collaborator Author

I can separate them if there's a preference, but it's difficult for me to test one without the other.

michaelkirk added a commit to michaelkirk/maplibre-navigation-ios that referenced this issue Jun 14, 2024
Previously, having voice instructions was a requirement of ending the
route.

Now we'll end the route when banner instructions are done *and* only
consider voice instructions if there were any to begin with.

Note: This currently crashes do to another issue with the end of route
view controller: see maplibre#57
michaelkirk added a commit to michaelkirk/maplibre-navigation-ios that referenced this issue Jun 20, 2024
Previously, having voice instructions was a requirement of ending the
route.

Now we'll end the route when banner instructions are done *and* only
consider voice instructions if there were any to begin with.

Note: This currently crashes do to another issue with the end of route
view controller: see maplibre#57
michaelkirk added a commit to michaelkirk/maplibre-navigation-ios that referenced this issue Jun 24, 2024
FIXES maplibre#57 - by rewriting EORVC as programmatic VC rather than storyboard.
FIXES maplibre#62 - Omitted the "Rate this route" feedback while rewriting the EORVC.

Note this was cherry-picked, and no longer works due to change in maplibre#54
michaelkirk added a commit to michaelkirk/maplibre-navigation-ios that referenced this issue Jun 27, 2024
FIXES maplibre#57 - by rewriting EORVC as programmatic VC rather than storyboard.
FIXES maplibre#62 - Omitted the "Rate this route" feedback while rewriting the EORVC.

Note this was cherry-picked, and no longer works due to change in maplibre#54
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 a pull request may close this issue.

1 participant