Skip to content

[7.x] [Reporting] Rewrite addForceNowQuerystring to getFullUrls (#44851)#45205

Merged
tsullivan merged 3 commits intoelastic:7.xfrom
tsullivan:backport/7.x/pr-44851
Sep 13, 2019
Merged

[7.x] [Reporting] Rewrite addForceNowQuerystring to getFullUrls (#44851)#45205
tsullivan merged 3 commits intoelastic:7.xfrom
tsullivan:backport/7.x/pr-44851

Conversation

@tsullivan
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

)

* this is the one

* this could break a job nbd

* get_full_url to observe PNG and PDF job payload types

* fix typescripts

* fix ts in tests

* fix more types

* cosmetic

* remove PDF execute compatibility shim test -- that stuff is handled in now PDF executeJob

* fix unit test

* remove old strings

* Remove pdf execute compatibility shim entirely

* combine the 2 maps

* More reject matchers in the test
@tsullivan tsullivan added the backport This PR is a backport of another PR label Sep 9, 2019
@tsullivan
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@spalger
Copy link
Copy Markdown
Contributor

spalger commented Sep 12, 2019

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@tsullivan tsullivan merged commit 9fda3f9 into elastic:7.x Sep 13, 2019
@tsullivan tsullivan deleted the backport/7.x/pr-44851 branch September 13, 2019 01:00
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