-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Updates publicBaseUrl warning copy #126296
Updates publicBaseUrl warning copy #126296
Conversation
Pinging @elastic/kibana-core (Team:Core) |
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. It'd be nice to have more explanation in the docs of the benefits of including this setting, as you suggested in #109970 (comment), but I think we should leave that to the alerting/reporting folks in a separate PR.
I like the positive spin you put on the title. In the UI, we typically say "Learn more" instead of "Refer to the documentation". |
@gchaps, thanks, I'll update that |
💚 Build SucceededMetrics [docs]Page load bundle
History
To update your PR or re-run it, just comment with: |
💔 Backport failedThe pull request could not be backported due to the following error: How to fixRe-run the backport manually:
Questions ?Please refer to the Backport tool documentation |
Friendly reminder: Looks like this PR hasn’t been backported yet. |
Friendly reminder: Looks like this PR hasn’t been backported yet. |
This should also have been backported to |
Summary
Resolves #109970.
Updates copy in the
publicBaseUrl
warning shown whenserver.publicBaseUrl
is not configured.Checklist
Delete any items that are not applicable to this PR.
Risk Matrix
For maintainers