-
Notifications
You must be signed in to change notification settings - Fork 131
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
Deprecate FollowPuckViewportStateOptions.animationDurationMs. #1256
Deprecate FollowPuckViewportStateOptions.animationDurationMs. #1256
Conversation
Co-authored-by: Ankur Khandelwal <ankur.khandelwal@mapbox.com>
@@ -60,6 +60,7 @@ class FollowPuckViewportStateOptions private constructor( | |||
* | |||
* Defaults to [DEFAULT_STATE_ANIMATION_DURATION_MS] milliseconds | |||
*/ | |||
@Deprecated("AnimationDurationMs is not needed any more, the transition will be handled properly internally.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume internally it's not handled for now - that's why we can't drop it right now, correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, the actual fix will land in rc.1, we don't want to rush the fix itself, Deprecate affects public API, so that's why we add the Deprecated
in beta.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with a question
Summary of changes
This PR deprecates
FollowPuckViewportStateOptions.animationDurationMs
, the transition will will handled properly by the Viewport plugin internally.User impact (optional)
Pull request checklist:
@JvmOverloads
,@file:JvmName
, etc).make update-api
to update generated api files, if there's public API changes, otherwise theverify-api-*
CI steps might fail.check changelog
CI step will fail.v10.[version]
release branch fix / enhancement, merge it tomain
firstly and then port tov10.[version]
release branch.Fixes: < Link to related issues that will be fixed by this pull request, if they exist >
PRs must be submitted under the terms of our Contributor License Agreement CLA.