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] Replace Legacy CSV Export flattenHit implementation with the shared implementation #115242

Closed
tsullivan opened this issue Oct 15, 2021 · 2 comments
Labels
bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Feature:Reporting:CSV Reporting issues pertaining to CSV file export impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort needs-team Issues missing a team label

Comments

@tsullivan
Copy link
Member

Kibana version: 7.16

Problem statement: There are too many flattenHit implementations in the Kibana code. The legacy csv export in Reporting should use the shared implementation that Discover uses.

This could solve #101232 for legacy CSV export. However, since legacy CSV is only available for users that have set it up in automation on an old version, we can assume there will not be any user-facing improvements by this change.

@tsullivan tsullivan added the bug Fixes for quality problems that affect the customer experience label Oct 15, 2021
@botelastic botelastic bot added the needs-team Issues missing a team label label Oct 15, 2021
@tsullivan tsullivan added impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. Team:AppServicesUx labels Oct 15, 2021
@elasticmachine
Copy link
Contributor

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

@botelastic botelastic bot removed the needs-team Issues missing a team label label Oct 15, 2021
@exalate-issue-sync exalate-issue-sync bot added the loe:small Small Level of Effort label Oct 18, 2021
@exalate-issue-sync exalate-issue-sync bot added loe:medium Medium Level of Effort and removed loe:small Small Level of Effort labels Oct 27, 2021
@exalate-issue-sync exalate-issue-sync bot added loe:small Small Level of Effort and removed loe:medium Medium Level of Effort labels Nov 22, 2021
@tsullivan
Copy link
Member Author

superceded by #121259

@sophiec20 sophiec20 added the Feature:Reporting:CSV Reporting issues pertaining to CSV file export label Aug 21, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Aug 21, 2024
@sophiec20 sophiec20 added (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead and removed (Deprecated) Team:Reporting Services labels Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Feature:Reporting:CSV Reporting issues pertaining to CSV file export impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort needs-team Issues missing a team label
Projects
None yet
Development

No branches or pull requests

3 participants