-
Notifications
You must be signed in to change notification settings - Fork 8.5k
[App Search] Temporarily remove sidebar layout for 7.10 release #79506
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
Conversation
- Since our other top level pages (settings, credentials, role mappings) aren't yet ready, we'll stick to the same UI for 7.9, only showing the Engines Overview to users
|
@elastic/enterprise-search-frontend Pinging for an approval since this is a time-sensitive PR that needs to be merged in by EOD today. The follow-up PR reverting the change will not be time-sensitive. Thank you! 🙇♀️ |
scottybollinger
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@elasticmachine merge upstream |
|
ignoring request to update branch, pull request is closed |
…ease (elastic#79506)" This reverts commit d5b8a95.
…ease (#79506)" (#79702) This reverts commit d5b8a95. Co-authored-by: Kibana Machine <[email protected]>
…ease (#79506)" (#79702) (#79959) This reverts commit d5b8a95. Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Kibana Machine <[email protected]>

Summary
Since our other top level pages (settings, credentials, role mappings) aren't yet ready, we're reverting to the 7.9 UI for 7.10 (i.e., only showing the Engines Overview to users).
Screencaps
QA
Checklist
Please note: this commit will immediately be reverted for 7.x/7.11 once the 7.10 branch is cut (tomorrow/FF). At that point, this PR should only exist in the 7.10 branch.