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

Add delegates to NavigationViewController. #4175

Merged

Conversation

MaximAlien
Copy link
Contributor

Description

PR adds delegate methods to NavigationViewController that allow to detect taps on waypoints and continuous alternatives.

@MaximAlien MaximAlien added this to the v2.9 milestone Sep 27, 2022
@MaximAlien MaximAlien requested a review from a team September 27, 2022 04:50
@MaximAlien MaximAlien self-assigned this Sep 27, 2022
@MaximAlien MaximAlien force-pushed the maxim/NAVIOS-614-add-delegates-to-navigation-view-controller branch from d4b4cbf to ab15b2b Compare September 27, 2022 04:58
@mapbox-github-ci-issues-public-1

Breaking Changes in MapboxNavigation

Breaking API Changes

NavigationViewControllerDelegate

  • removed method: navigationViewController(_:didSelect:) in NavigationViewControllerDelegate

/**
`UnimplementedLogging` prints a warning to standard output the first time this method is called.
*/
func navigationViewController(_ navigationViewController: NavigationViewController, didSelect route: Route) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Such method is not actually exposed anywhere, can be removed.

Copy link
Contributor

Choose a reason for hiding this comment

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

Should we expose it then? Or it is outdated and non existent delegate method?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I do not think that anybody requested it recently, so we can add it when it's really needed.

@mapbox-github-ci-issues-public-1

Breaking Changes in MapboxNavigation

Breaking API Changes

NavigationViewControllerDelegate

  • removed method: navigationViewController(_:didSelect:) in NavigationViewControllerDelegate

@mapbox-github-ci-issues-public-1

Breaking Changes in MapboxNavigation

Breaking API Changes

NavigationViewControllerDelegate

  • removed method: navigationViewController(_:didSelect:) in NavigationViewControllerDelegate

@MaximAlien MaximAlien merged commit a915d1c into main Sep 30, 2022
@MaximAlien MaximAlien deleted the maxim/NAVIOS-614-add-delegates-to-navigation-view-controller branch September 30, 2022 16:27
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.

None yet

2 participants