Skip to content

[7.x] [Enterprise Search] Remove all instances of KibanaContext to Kea store (#78513)#78692

Merged
cee-chen merged 1 commit intoelastic:7.xfrom
cee-chen:backport/7.x/pr-78513
Sep 28, 2020
Merged

[7.x] [Enterprise Search] Remove all instances of KibanaContext to Kea store (#78513)#78692
cee-chen merged 1 commit intoelastic:7.xfrom
cee-chen:backport/7.x/pr-78513

Conversation

@cee-chen
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

elastic#78513)

* Add KibanaLogic store/mount

* Update usage of setBreadcrumbs/setDocTitle to KibanaLogic

* Update usage of Kibana config context to KibanaLogic

* Update usage of navigateToUrl context to KibanaLogic

* 🔥 Remove unused shallow_usecontext mocks, repurpose file to shallow_useeffect

- The file is now no longer used to mock useContext, only unseEffect, hence the rename

* 🔥 Delete/repurpose mount with context helpers

- Remove mountWithKibanaContext completely
- Change mountWithContext to mountWithIntl, since that's the only context remaining, and move it to its own file
- Change mountWithAsyncContext to just mountAsync and move it to its own file
  + add an option to pair it w/ mountWithIntl for formatted date/number support

* 🔥 Delete KibanaContext and mockKibanaContext

+ minor newline linting/grouping tweaks

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@cee-chen cee-chen added the backport This PR is a backport of another PR label Sep 28, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id value diff baseline
enterpriseSearch 249 +2 247

async chunks size

id value diff baseline
enterpriseSearch 471.7KB -16.8KB 488.4KB

page load bundle size

id value diff baseline
enterpriseSearch 21.7KB +3.0B 21.7KB

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

@cee-chen cee-chen merged commit aa4dc72 into elastic:7.x Sep 28, 2020
@cee-chen cee-chen deleted the backport/7.x/pr-78513 branch September 28, 2020 22:58
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