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

TSVB needs to display better UX message when default index pattern is non-time based #124341

Merged
merged 2 commits into from
Feb 8, 2022

Conversation

DianaDerevyankina
Copy link
Contributor

@DianaDerevyankina DianaDerevyankina commented Feb 2, 2022

Closes #123863

Summary

Display an error message in case the time field wasn't specified.

image

For maintainers

@DianaDerevyankina DianaDerevyankina self-assigned this Feb 2, 2022
@DianaDerevyankina DianaDerevyankina changed the title TSVB needs to display better UX message when default index pattern is… TSVB needs to display better UX message when default index pattern is non-time based Feb 2, 2022
@DianaDerevyankina DianaDerevyankina added Feature:TSVB TSVB (Time Series Visual Builder) release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure v8.2.0 labels Feb 7, 2022
@alexwizp
Copy link
Contributor

alexwizp commented Feb 7, 2022

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

cc @dziyanadzeraviankina

@@ -34,7 +35,11 @@ export function getIntervalAndTimefield(
}

if (panel.use_kibana_indexes) {
validateField(timeField!, index);
if (timeField) {
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@alexwizp alexwizp left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you

@alexwizp alexwizp added the backport:skip This commit does not require backporting label Feb 7, 2022
@alexwizp alexwizp marked this pull request as ready for review February 7, 2022 12:44
@alexwizp alexwizp requested a review from a team as a code owner February 7, 2022 12:44
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

@flash1293 flash1293 left a comment

Choose a reason for hiding this comment

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

Tested and works as expected, LGTM

@alexwizp alexwizp merged commit c299aab into elastic:main Feb 8, 2022
@DianaDerevyankina DianaDerevyankina removed the backport:skip This commit does not require backporting label Feb 8, 2022
DianaDerevyankina added a commit to DianaDerevyankina/kibana that referenced this pull request Feb 8, 2022
… non-time based (elastic#124341)

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit c299aab)
DianaDerevyankina added a commit that referenced this pull request Feb 8, 2022
… non-time based (#124341) (#124937)

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit c299aab)

Co-authored-by: Kibana Machine <[email protected]>
@drewdaemon
Copy link
Contributor

👏 as a new TSVB user, I ran into this several times.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:TSVB TSVB (Time Series Visual Builder) release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure v8.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TSVB needs to display better UX message when default index pattern is non-time based.
7 participants