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

[DOCS] Updates notifications lifetime advanced settings descriptions #97692

Conversation

TinaHeiligers
Copy link
Contributor

Summary

Updates the notifications lifetime settings description: Removes mention of 'Infinity' as an option.
Supporting documentation changes for #30109

Checklist

Delete any items that are not applicable to this PR.

  • Documentation was added for features that require explanation or tutorials

For maintainers

@TinaHeiligers TinaHeiligers added Team:Docs Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.0.0 release_note:skip Skip the PR/issue when compiling release notes docs v7.13.0 auto-backport Deprecated - use backport:version if exact versions are needed labels Apr 20, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs (Team:Docs)

@TinaHeiligers TinaHeiligers changed the title Updates notifications lifetime advanced settings descriptions [DOCS] Updates notifications lifetime advanced settings descriptions Apr 20, 2021
Copy link
Contributor

@gchaps gchaps left a comment

Choose a reason for hiding this comment

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

Text LGTM

@TinaHeiligers TinaHeiligers merged commit 4a8067b into elastic:master Apr 21, 2021
@TinaHeiligers TinaHeiligers deleted the docs/uisettings-notifications-lifetimes branch April 21, 2021 14:55
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

@spalger
Copy link
Contributor

spalger commented Apr 21, 2021

@TinaHeiligers please note that this was merged into 7.x after we branched 7.13, but before the backport config was updated. Please make sure this is backported to the 7.13 branch if intended or remove the 7.13 label from this PR

@TinaHeiligers
Copy link
Contributor Author

@spalger I took a chance on merging this because it's a docs PR. I backported to 7.13 manually, on the advice of the Docs team. Thanks for the heads up though!

@spalger
Copy link
Contributor

spalger commented Apr 21, 2021

Ah, missed that, thanks! f06be5f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed docs release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Team:Docs v7.13.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants