Skip to content

[7.x] [Reporting/Server] register plugin routes synchronously (#68976)#69224

Merged
tsullivan merged 2 commits intoelastic:7.xfrom
tsullivan:backport/7.x/pr-68976
Jun 16, 2020
Merged

[7.x] [Reporting/Server] register plugin routes synchronously (#68976)#69224
tsullivan merged 2 commits intoelastic:7.xfrom
tsullivan:backport/7.x/pr-68976

Conversation

@tsullivan
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

* register routes synchronously

* back out some refactoring

* comment fix

* fix tests

* register route handler context provider

* Add function level comments in core methods

* fix tests

* revert editor help

* route context is the ReportingStart contract

* Fix reporting job route tests

* Fix generation tests

Co-authored-by: Joel Griffith <joel.griffith@elastic.co>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
# Conflicts:
#	x-pack/plugins/reporting/server/routes/generation.ts
@tsullivan tsullivan added the backport This PR is a backport of another PR label Jun 15, 2020
@tsullivan
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

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

@tsullivan tsullivan merged commit 0228ad7 into elastic:7.x Jun 16, 2020
@tsullivan tsullivan deleted the backport/7.x/pr-68976 branch June 16, 2020 22:18
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