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

[7.x] Improve deprecation message for xpack.spaces.enabled #112242

Merged
merged 14 commits into from
Oct 6, 2021

Conversation

watson
Copy link
Contributor

@watson watson commented Sep 15, 2021

Fixes part of #82467

This PR improves the deprecation message for if xpack.spaces.enabled is set to false in kibana.yml. This message also currently exists in master. However, I'll make another PR to remove it from master as this is going away in 8.0.

Here's how the new message will look in the Upgrade Assistant UI:

image

@watson watson added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting v7.16.0 labels Sep 15, 2021
@watson watson requested review from a team September 15, 2021 11:41
@watson watson self-assigned this Sep 15, 2021
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
@watson
Copy link
Contributor Author

watson commented Sep 21, 2021

@elasticmachine merge upstream

@watson
Copy link
Contributor Author

watson commented Sep 22, 2021

@elasticmachine merge upstream

@watson
Copy link
Contributor Author

watson commented Sep 28, 2021

@elasticmachine merge upstream

Copy link
Contributor

@jportner jportner left a comment

Choose a reason for hiding this comment

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

Thanks for updating this, LGTM.

Here is the current screenshot:

image

As discussed in #112242 (comment), there is a separate issue to render a single-step corrective action as a paragraph instead of a list.

Copy link
Contributor

@debadair debadair left a comment

Choose a reason for hiding this comment

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

A suggested edit that I think is more in line with the guidelines.

x-pack/plugins/spaces/server/config.ts Outdated Show resolved Hide resolved
Co-authored-by: debadair <debadair@elastic.co>
@watson
Copy link
Contributor Author

watson commented Oct 6, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

cc @watson

@watson watson merged commit e17077c into elastic:7.x Oct 6, 2021
@watson watson deleted the deprecation-spaces-enabled branch October 6, 2021 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes v7.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants