Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[ios, macos] Deprecate methods in favor of new completion handler versions #14959

Merged

Conversation

friedbunny
Copy link
Contributor

Deprecates various MGLMapView methods and replaces them with completion hander versions being added in #14381.

This follows our best practices for deprecation, as outlined in #14560 (comment).

This will need changelogging and currently targets the #14381 branch — we should figure out the changelog when we merge that and retarget this PR.

/cc @1ec5 @julianrex

@friedbunny friedbunny added iOS Mapbox Maps SDK for iOS refactor macOS Mapbox Maps SDK for macOS labels Jun 18, 2019
@friedbunny friedbunny added this to the release-p milestone Jun 18, 2019
@friedbunny friedbunny requested review from 1ec5 and a team June 18, 2019 19:22
@friedbunny friedbunny self-assigned this Jun 18, 2019
@friedbunny friedbunny added the needs changelog Indicates PR needs a changelog entry prior to merging. label Jun 19, 2019
platform/ios/src/MGLMapView.h Show resolved Hide resolved
platform/ios/src/MGLMapView.h Outdated Show resolved Hide resolved
@friedbunny friedbunny force-pushed the 1ec5-animation-completion-5839 branch 2 times, most recently from 082aa34 to 87fa146 Compare June 22, 2019 00:09
@friedbunny friedbunny force-pushed the friedbunny-deprecates-uncompletionful-methods-5839 branch 2 times, most recently from 9d0802a to 6ed7420 Compare June 22, 2019 00:42
@friedbunny friedbunny changed the base branch from 1ec5-animation-completion-5839 to master June 22, 2019 00:46
@friedbunny friedbunny force-pushed the friedbunny-deprecates-uncompletionful-methods-5839 branch from 6ed7420 to 45b990d Compare June 22, 2019 00:49
@friedbunny friedbunny removed the needs changelog Indicates PR needs a changelog entry prior to merging. label Jun 22, 2019
@friedbunny friedbunny requested review from 1ec5 and julianrex June 22, 2019 00:49
@friedbunny friedbunny force-pushed the friedbunny-deprecates-uncompletionful-methods-5839 branch from 45b990d to 8a9567e Compare June 26, 2019 19:18
@friedbunny
Copy link
Contributor Author

Rebased to bring in the new render tests on CI.

Copy link
Contributor

@julianrex julianrex left a comment

Choose a reason for hiding this comment

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

LGTM - thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants