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

[Lens] Do not submit invalid query in filtered metric #95611

Closed
flash1293 opened this issue Mar 27, 2021 · 1 comment · Fixed by #107542
Closed

[Lens] Do not submit invalid query in filtered metric #95611

flash1293 opened this issue Mar 27, 2021 · 1 comment · Fixed by #107542
Assignees
Labels
enhancement New value added to drive a business result Feature:Lens good first issue low hanging fruit Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@flash1293
Copy link
Contributor

If the query of a filtered metric is invalid, it's still attempting to render the chart which is causing a runtime error:
Screenshot 2021-03-27 at 14 33 38

This should be improved. In case of the filters function an invalid query is implicitly turned into a * query matching all documents - we could do the same thing here, but maybe we should do something more sensible in both cases like not submitting the query as long as the input is open and on close failing the validation and highlighting the problematic dimension.

@flash1293 flash1293 added enhancement New value added to drive a business result Team:Visualizations Visualization editors, elastic-charts and infrastructure Feature:Lens labels Mar 27, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@flash1293 flash1293 added the good first issue low hanging fruit label Jun 17, 2021
alexwizp added a commit to alexwizp/kibana that referenced this issue Aug 3, 2021
alexwizp added a commit that referenced this issue Aug 11, 2021
* [Lens] Do not submit invalid query in filtered metric

Closes: #95611

* fix CI

* fix PR comments

* fix PR comments

* fix PR comment

* move closePopover to useCallback

* add filter validation to utils/isColumnInvalid

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
alexwizp added a commit to alexwizp/kibana that referenced this issue Aug 11, 2021
…7542)

* [Lens] Do not submit invalid query in filtered metric

Closes: elastic#95611

* fix CI

* fix PR comments

* fix PR comments

* fix PR comment

* move closePopover to useCallback

* add filter validation to utils/isColumnInvalid

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
alexwizp added a commit that referenced this issue Aug 11, 2021
…108144)

* [Lens] Do not submit invalid query in filtered metric

Closes: #95611

* fix CI

* fix PR comments

* fix PR comments

* fix PR comment

* move closePopover to useCallback

* add filter validation to utils/isColumnInvalid

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Lens good first issue low hanging fruit Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants