Skip to content

[7.x] [App Search] Implement initial Engine routing/navigation (#82549)#82758

Merged
cee-chen merged 1 commit intoelastic:7.xfrom
cee-chen:backport/7.x/pr-82549
Nov 5, 2020
Merged

[7.x] [App Search] Implement initial Engine routing/navigation (#82549)#82758
cee-chen merged 1 commit intoelastic:7.xfrom
cee-chen:backport/7.x/pr-82549

Conversation

@cee-chen
Copy link
Copy Markdown
Contributor

@cee-chen cee-chen commented Nov 5, 2020

Backports the following commits to 7.x:

* [Setup] Update routes + role privileges

Routes: use generatePath to better match ent-search
Roles: We're using "Search UI" in the nav copy now, so we should update our vars accordingly

* Add new EngineNav and EngineRouter components

* Update existing components to use new EngineRouter/EngineNav

* Add App Search engine label & new SideNavItem component

* Add EngineRouter breadcrumbs

* [Refactor] DRY out i18n constants
@cee-chen cee-chen added the backport This PR is a backport of another PR label Nov 5, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
enterpriseSearch 438 450 +12

async chunks size

id before after diff
enterpriseSearch 652.8KB 666.5KB +13.7KB

distributable file count

id before after diff
default 43079 43080 +1

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

@cee-chen cee-chen merged commit 6adc11a into elastic:7.x Nov 5, 2020
@cee-chen cee-chen deleted the backport/7.x/pr-82549 branch November 5, 2020 20:17
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