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

[Timelion] Hide fit from suggestions #119568

Merged
merged 2 commits into from
Nov 27, 2021

Conversation

flash1293
Copy link
Contributor

Fixes #8460

This PR makes sure the fit argument of the .es function isn't advertised to the user as it doesn't have a meaningful function in the current feature-set. Other datasources (graphite) continue to behave the same way, as well as legacy timelion charts which define this argument already

Before:
Screenshot 2021-11-24 at 10 17 11

After:
Screenshot 2021-11-24 at 10 14 23

@flash1293 flash1293 added Feature:Timelion Timelion app and visualization release_note:fix Team:Visualizations Visualization editors, elastic-charts and infrastructure backport:skip This commit does not require backporting v8.1.0 labels Nov 24, 2021
@flash1293 flash1293 marked this pull request as ready for review November 24, 2021 10:41
@flash1293 flash1293 requested a review from a team as a code owner November 24, 2021 10:41
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors)

@stratoula
Copy link
Contributor

@flash1293 if I understand correctly from the description, the fit function should be available for the flot implementation, right? I don't see it in the list when the advanced setting is on.
image

Copy link
Contributor

@stratoula stratoula left a comment

Choose a reason for hiding this comment

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

Update: I read the issue more carefully and your comment here. LGTM! I tested it locally and the fit function is hidden from the autocomplete.

@flash1293
Copy link
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
visTypeTimelion 88.0KB 88.0KB +11.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@flash1293 flash1293 merged commit 6367b17 into elastic:main Nov 27, 2021
TinLe pushed a commit to TinLe/kibana that referenced this pull request Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting Feature:Timelion Timelion app and visualization release_note:fix Team:Visualizations Visualization editors, elastic-charts and infrastructure v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Kibana] [Timelion] ES Datasource Does Not Fit Null Values
5 participants