-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Spaces] Remove forceSolutionVisibility yml setting #204726
[Spaces] Remove forceSolutionVisibility yml setting #204726
Conversation
/ci |
💚 Build Succeeded
Metrics [docs]
cc @sebelga |
Pinging @elastic/appex-sharedux (Team:SharedUX) |
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, thanks
Starting backport for target branches: 8.x https://github.com/elastic/kibana/actions/runs/12397694916 |
(cherry picked from commit cd3c5b6)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…204805) # Backport This will backport the following commits from `main` to `8.x`: - [[Spaces] Remove forceSolutionVisibility yml setting (#204726)](#204726) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Sébastien Loix","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-18T16:58:47Z","message":"[Spaces] Remove forceSolutionVisibility yml setting (#204726)","sha":"cd3c5b6704ce5c97f272e4552c4cfa81cc58f630","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Security/Spaces","release_note:skip","v9.0.0","Team:SharedUX","backport:prev-minor"],"title":"[Spaces] Remove forceSolutionVisibility yml setting","number":204726,"url":"https://github.com/elastic/kibana/pull/204726","mergeCommit":{"message":"[Spaces] Remove forceSolutionVisibility yml setting (#204726)","sha":"cd3c5b6704ce5c97f272e4552c4cfa81cc58f630"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/204726","number":204726,"mergeCommit":{"message":"[Spaces] Remove forceSolutionVisibility yml setting (#204726)","sha":"cd3c5b6704ce5c97f272e4552c4cfa81cc58f630"}}]}] BACKPORT--> Co-authored-by: Sébastien Loix <[email protected]>
In #191743 we introduced a temporary yml setting for the security team so they could enable the new side navigation in their internal deployment.
Since #203239 was merged we don't need this yml setting anymore. We can safely remove it as it was only ever used internally and I checked with the security team to make sure they are not using this setting in their deployment.
Fixes #193132