-
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
[Alerting] Should we use custom deprecated messages for removing .enabled
config?
#114190
Comments
Pinging @elastic/kibana-alerting-services (Team:Alerting Services) |
Wondering if we want to add some doc on the new enabled/disabled config going away, and how to tell users to perform equivalent functionality. For instance, using feature controls to make sure no one has access to the feature ... We could leave the message as is, and customize it later if we find it's confusing users - by including a link to the docs, for instance. |
I doubt any other plugins have anything like that in their doc, but alerting is special! :-) I believe we did recently update a logged message to include a link to the doc, since it remediation wasn't something we'd really want in just an error message - can't remember what message that was though. I was thinking of it more as a way to not have to bake details into the messages, but instead bake it into the doc, since it's probably a good thing to doc anyway (hmmm, didn't it just come up in an internal conversation? :-) ) |
This was the thread we had back in August around linking to documentation on the server side: #109741 (comment) 🙂 The limitation was around not knowing the documentation URL + version suffix. But the thread explored a way to work around it for now. |
It looks like |
Closed with #120151 |
Relates to #114188
In #114188, the core team is advising plugin authors to ensure the default messaging around the
.enabled
deprecation is the best UX. If not, they are recommending to use custom messaging with examples of how to do that.The text was updated successfully, but these errors were encountered: