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.

Sign up for GitHub

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

Jump to bottom

[Lens] Fix issue downloading tables to CSV that contain "Split metrics by" #187551

Closed
teresaalvarezsoler opened this issue Jul 4, 2024 · 1 comment · Fixed by #193780
Closed

[Lens] Fix issue downloading tables to CSV that contain "Split metrics by" #187551

teresaalvarezsoler opened this issue Jul 4, 2024 · 1 comment · Fixed by #193780
Assignees
@nickofthyme
Labels
Feature:Lens impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:medium Medium Level of Effort Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@teresaalvarezsoler
Copy link

teresaalvarezsoler commented Jul 4, 2024 •
edited by nickofthyme
Loading

Problem
We introduced a change in 7.17 (#128297) for exporting tables to CSV that fixed some bugs for normal tables but it introduced a problem when the table is built using "Split metrics by". The issue is described here #164055

Although it's not technically a bug, customers are complaining about this and they would like to get exactly what they see (see https://github.com/elastic/sdh-kibana/issues/4757)

Solution

  • When the user downloads a table as CSV, we should default back to the previous export functionality ONLY if the table is built using "Split metrics by" so the user gets in the CSV the exact columns that he sees in the table.
  • We will add a third option if the user downloads the CSV from "Inspect", that way we still offer the current functionality along with the new one. Image
@teresaalvarezsoler teresaalvarezsoler added the Team:Visualizations Visualization editors, elastic-charts and infrastructure label Jul 4, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-visualizations (Team:Visualizations)

@teresaalvarezsoler teresaalvarezsoler changed the title [Lens] Fix issue downloading tables to CSV that contain "Split by" metrics [Lens] Fix issue downloading tables to CSV that contain "Split metrics by" Jul 4, 2024
@teresaalvarezsoler teresaalvarezsoler added impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:medium Medium Level of Effort and removed triage_needed labels Jul 10, 2024
@nickofthyme nickofthyme self-assigned this Sep 17, 2024
@nickofthyme nickofthyme mentioned this issue Sep 23, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nickofthyme nickofthyme

Labels
Feature:Lens impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:medium Medium Level of Effort Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Lens][Datatable] Fix share export and inspect data nickofthyme/kibana
3 participants
@elasticmachine @nickofthyme @teresaalvarezsoler