Skip to content

[7.x] [ui/utils/query_string]: Remove unused methods & migrate apps to querystring lib (#56957)#57533

Merged
alexwizp merged 1 commit intoelastic:7.xfrom
alexwizp:backport/7.x/pr-56957
Feb 13, 2020
Merged

[7.x] [ui/utils/query_string]: Remove unused methods & migrate apps to querystring lib (#56957)#57533
alexwizp merged 1 commit intoelastic:7.xfrom
alexwizp:backport/7.x/pr-56957

Conversation

@alexwizp
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

@alexwizp alexwizp added the backport This PR is a backport of another PR label Feb 13, 2020
…ystring lib (elastic#56957)

* replace querystring (querystring-browser) -> query-string

* QueryString remove encode/decode methods

* remove query_string file

* remove querystring-browser from package.json

* add kibana_utils\url module

* cleanup

* update notice.txt

* fix merge conflict

* fix CI

* fix wrong import

* fix CI

* fix X-Pack firefox smoke test

* remove urlUtils.parseUrlQuery

* remove url.stringifyUrlQuery

* use url.encodeQuery

* Record<string, any> -> ParsedQuery

* Update src/plugins/console/public/application/containers/editor/legacy/console_editor/editor.tsx

Co-Authored-By: Luke Elmers <lukeelmers@gmail.com>

* add more tests for APM

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: Luke Elmers <lukeelmers@gmail.com>

# Conflicts:
#	x-pack/plugins/endpoint/public/applications/endpoint/store/alerts/middleware.ts
@alexwizp alexwizp force-pushed the backport/7.x/pr-56957 branch from fa7c85d to b333869 Compare February 13, 2020 07:45
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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

@alexwizp alexwizp merged commit f3104aa into elastic:7.x Feb 13, 2020
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