Skip to content

[7.x] [Enterprise Search] Add parseQueryParams helper (#83750)#83842

Merged
scottybollinger merged 1 commit intoelastic:7.xfrom
scottybollinger:backport/7.x/pr-83750
Nov 19, 2020
Merged

[7.x] [Enterprise Search] Add parseQueryParams helper (#83750)#83842
scottybollinger merged 1 commit intoelastic:7.xfrom
scottybollinger:backport/7.x/pr-83750

Conversation

@scottybollinger
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* [Enterprise Search] Add parseQueryParams helper

This PR migrates part of the ent-search queryParams util, `parseQueryParams` for use in Workplace Search.

`setQueryParams` was no a part of this PR because it is only used one time in App Search and a better alternative might be available for that use-case

* Remove mock

* Actually test functionality of query-string

* Add test for array

* Better test name
@scottybollinger scottybollinger added the backport This PR is a backport of another PR label Nov 19, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@scottybollinger scottybollinger merged commit 02706ff into elastic:7.x Nov 19, 2020
@scottybollinger scottybollinger deleted the backport/7.x/pr-83750 branch November 19, 2020 21: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.

2 participants