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

[Reporting/Download CSV] Get the file name from savedSearch data #76031

Merged
merged 4 commits into from
Aug 27, 2020

Conversation

tsullivan
Copy link
Member

@tsullivan tsullivan commented Aug 26, 2020

Summary

Closes #75456

The function used for getting the panel title was returning an empty string when the Dashboard was configured to hide the panel titles. This PR changes which function is called to get the title of the saved search that is used as the downloaded file name.

Release note: fix a bug where the downloading CSV from a saved search in a dashboard panel had no file name if the dashboard panels were hidden.

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@elasticmachine
Copy link
Contributor

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

@tsullivan tsullivan requested a review from a team as a code owner August 26, 2020 22:16
@botelastic botelastic bot added the Feature:Embedding Embedding content via iFrame label Aug 26, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

page load bundle size

id value diff baseline
embeddable 430.5KB +51.0B 430.5KB
reporting 290.8KB +12.0B 290.8KB
total +63.0B

History

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

Copy link
Contributor

@lizozom lizozom 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 LGTM

@tsullivan tsullivan merged commit 69a8d06 into elastic:master Aug 27, 2020
@tsullivan tsullivan deleted the fix/75456 branch August 27, 2020 15:41
tsullivan added a commit to tsullivan/kibana that referenced this pull request Aug 27, 2020
…stic#76031)

* [Reporting/Download CSV] provide title even if panel \titles are hidden in the dashboard

* add functional test

* Update embeddable_panel.tsx

* Update download_csv.ts
tsullivan added a commit that referenced this pull request Aug 28, 2020
) (#76104)

* [Reporting/Download CSV] provide title even if panel \titles are hidden in the dashboard

* add functional test

* Update embeddable_panel.tsx

* Update download_csv.ts
@sophiec20 sophiec20 added the (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Feature:Embedding Embedding content via iFrame release_note:fix v7.10.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Reporting] CSV export from panel does not show a name if panel title is hidden
5 participants