Skip to content

Conversation

@ferullo
Copy link
Contributor

@ferullo ferullo commented Aug 1, 2025

In my last PR to this file I missed a few places clarification was needed. I found them when going through this file again to align Kibana tool tips to it.

@ferullo ferullo requested a review from a team as a code owner August 1, 2025 15:37
@github-actions
Copy link

github-actions bot commented Aug 1, 2025

🔍 Preview links for changed docs

@ferullo
Copy link
Contributor Author

ferullo commented Aug 1, 2025

Thanks @natasha-moore-elastic ! I'll merge this when I merge the Kibana PR so I don't come back with another PR to this file 😄

@florent-leborgne
Copy link
Contributor

Hey @ferullo, are we good to merge this PR? Asking since it's 1,5 month old now :)

@ferullo
Copy link
Contributor Author

ferullo commented Sep 22, 2025

I was waiting to merge a Kibana PR at the same time. But that PR is going to be delayed even longer. This can merge now.

@ferullo ferullo enabled auto-merge (squash) September 22, 2025 13:14
@ferullo ferullo merged commit 46a4230 into main Sep 22, 2025
6 of 7 checks passed
@ferullo ferullo deleted the ferullo-defend-advanced-options-2 branch September 22, 2025 13:17
ferullo added a commit to elastic/kibana that referenced this pull request Nov 12, 2025
…entation (#230269)

## Summary

This aligns the tool-tips shown in Kibana to the same
Docs-Team-approved-text on
https://www.elastic.co/docs/reference/security/defend-advanced-settings
(including a few pending updates in
elastic/docs-content#2365). One new advanced
option is added. It's been missing from Kibana -- it isn't new.

### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)

### Identify risks

No risk assuming this renders right in Kibana. It's just tool-tip
wording changes.

---------

Co-authored-by: Ash <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
…entation (elastic#230269)

## Summary

This aligns the tool-tips shown in Kibana to the same
Docs-Team-approved-text on
https://www.elastic.co/docs/reference/security/defend-advanced-settings
(including a few pending updates in
elastic/docs-content#2365). One new advanced
option is added. It's been missing from Kibana -- it isn't new.

### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)

### Identify risks

No risk assuming this renders right in Kibana. It's just tool-tip
wording changes.

---------

Co-authored-by: Ash <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants