-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Maps] Remove deprecated usage of indexPattern.title #143444
Conversation
Pinging @elastic/kibana-gis (Team:Geo) |
@elasticmachine merge upstream |
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.
Code changes looks good to me, thanks for taking care of the deprecation. I've checked locally for any other usage of .title
for index patterns or data views but looks like everything is addressed 👍
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 - thanks for cleaning up deprecated use of dataView.title and +1 on renames.
code review
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |
Summary
Fixes deprecated data view calls.
Accessing
indexPattern.title
directly has been deprecated and should be replaced by thegetIndexPattern
method. In many cases I changed the local variable todataView
because callingindexPattern.getIndexPattern()
may be confusing.We could do much more with replacing
indexPattern
withdataView
in other parts of the code, but I kept this PR small so that we only need to fix the deprecations.