Skip to content

[7.x] Improve search typescript (#69333)#71021

Merged
lizozom merged 4 commits intoelastic:7.xfrom
lizozom:backport/7.x/pr-69333
Jul 8, 2020
Merged

[7.x] Improve search typescript (#69333)#71021
lizozom merged 4 commits intoelastic:7.xfrom
lizozom:backport/7.x/pr-69333

Conversation

@lizozom
Copy link
Copy Markdown
Contributor

@lizozom lizozom commented Jul 7, 2020

Backports the following commits to 7.x:

* [search] Refactor the way search strategies are registered/retrieved on the server

* Fix types and tests and update docs

* Fix failing test

* Fix build of example plugin

* Fix functional test

* Make server strategies sync

* Move strategy name into options

* docs

* Remove FE strategies

* TypeScript of hell
delete search explorer

* Fix search interceptor OSS tests

* typos

* test cleanup

* Delete search example
fix interceptor async tests to use fake timers

* docs

* fix

* return search wrapper

* Update search interceptor tests and abort utils

* ts

* jest test fix

* code review

* change how logs consume search API

Co-authored-by: Lukas Olson <olson.lukas@gmail.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
# Conflicts:
#	src/plugins/data/public/public.api.md
@lizozom lizozom added the backport This PR is a backport of another PR label Jul 7, 2020
@lizozom
Copy link
Copy Markdown
Contributor Author

lizozom commented Jul 8, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

@kbn/optimizer bundle module count

id value diff baseline
data 203 -2 205
dataEnhanced 5 -1 6
total - -3 -

History

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

@lizozom lizozom merged commit eda424a into elastic:7.x Jul 8, 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.

3 participants