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

Use visualization name for CSV export filename #67289

Closed
dave-wheeler opened this issue May 25, 2020 · 6 comments · Fixed by #71249
Closed

Use visualization name for CSV export filename #67289

dave-wheeler opened this issue May 25, 2020 · 6 comments · Fixed by #71249
Assignees
Labels
enhancement New value added to drive a business result Feature:Inspector Inspector infrastructure and implementations regression Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@dave-wheeler
Copy link

Kibana version: 7.7.0
Elasticsearch version: 7.7.0
Server OS version: Archlinux
Browser version: FF 76.0.1
Browser OS version: Archlinux
Original install method (e.g. download page, yum, from source, etc.): Download page/AUR package
Describe the bug: CSV export filename has regressed to always being table.csv.

Steps to reproduce:

  1. Open a visualization
  2. Click one of the export links (Raw or Formatted)
  3. File is named table.csv.

Expected behavior:
Visualization name is used for filename.

Any additional context:
#3625

@myasonik myasonik added bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Team:Reporting Services labels May 26, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

@tsullivan
Copy link
Member

Looks like this issue is about downloading the CSV from Inspector:

image

@tsullivan tsullivan added Team:Visualizations Visualization editors, elastic-charts and infrastructure and removed (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Team:Reporting Services labels May 27, 2020
@elasticmachine
Copy link
Contributor

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

@timroes timroes added enhancement New value added to drive a business result Feature:Inspector Inspector infrastructure and implementations and removed bug Fixes for quality problems that affect the customer experience labels May 27, 2020
@dave-wheeler
Copy link
Author

FYI: This worked as expected before 7.7, so it's really a regression bug, and not an enhancement or feature request.

@dave-wheeler
Copy link
Author

As I've mentioned earlier, this is a regression of a previously completed ticket, and not a feature request.
#3625

After having to change several scripts that use exported visualization data when #3625 was completed, I'm having to revert them with this regression (I've had to change them rename to table.csv to hard-coded-in-the-script visualization names). If this issue is ever addressed, I will have to change them a 3rd time. Is there an expected time frame for when that might be?

@dej611
Copy link
Contributor

dej611 commented Jul 8, 2020

I've managed to reproduce the issue, but only when using a new Data Table visualization, and clicking the Export links below the table.

Screenshot 2020-07-08 at 17 41 33

Using the Inspector download links the files are downloaded with as unsaved.csv filename.

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:Inspector Inspector infrastructure and implementations regression Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants