Skip to content

[7.x] [Reporting] Use spaceId from request in export generation (#76998)#77952

Merged
tsullivan merged 1 commit intoelastic:7.xfrom
tsullivan:backport/7.x/pr-76998
Sep 19, 2020
Merged

[7.x] [Reporting] Use spaceId from request in export generation (#76998)#77952
tsullivan merged 1 commit intoelastic:7.xfrom
tsullivan:backport/7.x/pr-76998

Conversation

@tsullivan
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

)

* [Reporting] Use spaceId from request in export generation

* remove todo that has been done

* whitespace

* use post params api in test

* add logging to core

* Update x-pack/plugins/reporting/server/export_types/printable_pdf/lib/get_custom_logo.ts

Co-authored-by: Joel Griffith <joel@joelgriffith.net>

* more logging

* fix interdependence and remove Promise.all

* getAbsoluteUrl have only 1 way to provide basePath

* --wip-- [skip ci]

* log apipath

* deleteAllReports at the end

* tests pass locally

* set config in the tests

* re-add skips of flaky tests

* test using csv:quoteValues

Co-authored-by: Joel Griffith <joel@joelgriffith.net>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@tsullivan tsullivan added the backport This PR is a backport of another PR label Sep 18, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

distributable file count

id value diff baseline
default 47620 -1 47621

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

@tsullivan tsullivan merged commit 97af0fc into elastic:7.x Sep 19, 2020
@tsullivan tsullivan deleted the backport/7.x/pr-76998 branch September 19, 2020 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants