Skip to content

[7.x] [Reporting] Define shims of legacy dependencies (#54082)#55134

Merged
tsullivan merged 5 commits intoelastic:7.xfrom
tsullivan:backport/7.x/pr-54082
Jan 21, 2020
Merged

[7.x] [Reporting] Define shims of legacy dependencies (#54082)#55134
tsullivan merged 5 commits intoelastic:7.xfrom
tsullivan:backport/7.x/pr-54082

Conversation

@tsullivan
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

* simplify serverfacade definition

* simplify requestfacade definition

* use the shim

* makeRequestFacade

* requestFacade

* import sorting

* originalServer

* reduce loc change

* remove consolelog

* hacks to fix tests

* ServerFacade in index

* Cosmetic

* remove field from serverfacade

* add raw to the request

* fix types

* add fieldFormatServiceFactory to legacy

* Pass the complete request object to sec plugin

* Fix test

* fix test 2

* getUser takes a legacy request

* add unit test for new lib

* add getRawRequest to pass to saved objects method

* update test snapshot

* leave a TODO comment for type import

* variable rename for legacy id

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@tsullivan tsullivan added the backport This PR is a backport of another PR label Jan 16, 2020
@tsullivan
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

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

@tsullivan tsullivan merged commit 1cb96c2 into elastic:7.x Jan 21, 2020
@tsullivan tsullivan deleted the backport/7.x/pr-54082 branch January 21, 2020 18:04
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.

3 participants