Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Search] [Playground] SideNav: move playground to build #181087

Merged
merged 12 commits into from Apr 22, 2024

Conversation

joemcelroy
Copy link
Member

@joemcelroy joemcelroy commented Apr 17, 2024

Update Search nav to build and move playground from content to build

image

update the kibana side nav to feature Playground. This routes from application to playground.

image

@joemcelroy joemcelroy added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting Team:EnterpriseSearch v8.14.0 labels Apr 17, 2024
@joemcelroy joemcelroy removed the backport:skip This commit does not require backporting label Apr 18, 2024
@joemcelroy joemcelroy marked this pull request as ready for review April 18, 2024 15:25
@joemcelroy joemcelroy requested review from a team as code owners April 18, 2024 15:25
@joemcelroy joemcelroy changed the title [Search] [Playground] move playground to applications, update links [Search] [Playground] SideNav: move playground to build Apr 18, 2024
@joemcelroy
Copy link
Member Author

@elasticmachine merge upstream

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty sure the changes here are why GitHub pinged me for review--LGTM

@joemcelroy joemcelroy requested a review from a team as a code owner April 19, 2024 13:17
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
enterpriseSearch 2268 2269 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
enterpriseSearch 2.7MB 2.7MB +388.0B
lists 141.0KB 141.1KB +33.0B
total +421.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
core 406.1KB 406.1KB +33.0B
enterpriseSearch 48.5KB 48.7KB +164.0B
navigation 30.2KB 30.2KB +5.0B
total +202.0B

History

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

@joemcelroy joemcelroy merged commit 019dd79 into elastic:main Apr 22, 2024
18 checks passed
@joemcelroy joemcelroy deleted the update-sidenav branch April 22, 2024 15:07
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Apr 22, 2024
Update Search nav to build and move playground from content to build

![image](https://github.com/elastic/kibana/assets/49480/8393a3e1-0d42-48c7-aa41-a9cc17ef48fa)

update the kibana side nav to feature Playground. This routes from
application to playground.

![image](https://github.com/elastic/kibana/assets/49480/22fe95df-e277-4c0b-8e65-edba8ba940cf)

---------

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 019dd79)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.14

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 23, 2024
joemcelroy added a commit that referenced this pull request Apr 25, 2024
…) (#181334)

# Backport

This will backport the following commits from `main` to `8.14`:
- [[Search] [Playground] SideNav: move playground to build
(#181087)](#181087)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Joe
McElroy","email":"joseph.mcelroy@elastic.co"},"sourceCommit":{"committedDate":"2024-04-22T15:07:35Z","message":"[Search]
[Playground] SideNav: move playground to build (#181087)\n\nUpdate
Search nav to build and move playground from content to
build\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/49480/8393a3e1-0d42-48c7-aa41-a9cc17ef48fa)\r\n\r\nupdate
the kibana side nav to feature Playground. This routes
from\r\napplication to
playground.\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/49480/22fe95df-e277-4c0b-8e65-edba8ba940cf)\r\n\r\n---------\r\n\r\nCo-authored-by:
Kibana Machine
<42973632+kibanamachine@users.noreply.github.com>","sha":"019dd790965ac86a3bb3f49619e41430aaa89b4e","branchLabelMapping":{"^v8.15.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:EnterpriseSearch","v8.14.0","v8.15.0"],"title":"[Search]
[Playground] SideNav: move playground to
build","number":181087,"url":"#181087
[Playground] SideNav: move playground to build (#181087)\n\nUpdate
Search nav to build and move playground from content to
build\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/49480/8393a3e1-0d42-48c7-aa41-a9cc17ef48fa)\r\n\r\nupdate
the kibana side nav to feature Playground. This routes
from\r\napplication to
playground.\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/49480/22fe95df-e277-4c0b-8e65-edba8ba940cf)\r\n\r\n---------\r\n\r\nCo-authored-by:
Kibana Machine
<42973632+kibanamachine@users.noreply.github.com>","sha":"019dd790965ac86a3bb3f49619e41430aaa89b4e"}},"sourceBranch":"main","suggestedTargetBranches":["8.14"],"targetPullRequestStates":[{"branch":"8.14","label":"v8.14.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.15.0","branchLabelMappingKey":"^v8.15.0$","isSourceBranch":true,"state":"MERGED","url":"#181087
[Playground] SideNav: move playground to build (#181087)\n\nUpdate
Search nav to build and move playground from content to
build\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/49480/8393a3e1-0d42-48c7-aa41-a9cc17ef48fa)\r\n\r\nupdate
the kibana side nav to feature Playground. This routes
from\r\napplication to
playground.\r\n\r\n\r\n![image](https://github.com/elastic/kibana/assets/49480/22fe95df-e277-4c0b-8e65-edba8ba940cf)\r\n\r\n---------\r\n\r\nCo-authored-by:
Kibana Machine
<42973632+kibanamachine@users.noreply.github.com>","sha":"019dd790965ac86a3bb3f49619e41430aaa89b4e"}}]}]
BACKPORT-->

Co-authored-by: Joe McElroy <joseph.mcelroy@elastic.co>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:EnterpriseSearch v8.14.0 v8.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants