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
all: document valid initial interval units. #6706
Conversation
35697e7
to
c819346
Compare
🌐 Coverage report
|
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
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.
- Should this be extended to cover
interval
too? - Can you please post a screenshot of one of the text fields with the description. I'm debating on whether the
NOTE:
is too strong visually vs havings/NOTE: S/The s/g
. I thinking that if it will appear on more than one field that the non-"NOTE" version might be better.
I'll add the interval too. For the formatting, the text here is taken from an existing integration (I don't recall which). In the wild, there are two dominant forms, the one here and one "NOTE:- Supported units for this parameter are h/m/s." |
c819346
to
28ef0c2
Compare
Not all acceptable units are listed, but rather choose commonly used units that are already documented in other packages. akamai, atlassian_bitbucket, atlassian_confluence, atlassian_jira, bitdefender, box_events, cisco_secure_endpoint, cloudflare, cloudflare, forgerock, github, github, jumpcloud, m365_defender, microsoft_defender_endpoint, microsoft_exchange_online_message_trace, mimecast, okta, panw_cortex_xdr, slack, ti_abusech, ti_cif3, ti_cybersixgill, ti_cybersixgill, ti_misp, ti_otx, ti_threatq, tines, zerofox, zeronetworks
28ef0c2
to
e8a03b7
Compare
Package akamai - 2.11.0 containing this change is available at https://epr.elastic.co/search?package=akamai |
Package atlassian_bitbucket - 1.12.0 containing this change is available at https://epr.elastic.co/search?package=atlassian_bitbucket |
Package atlassian_confluence - 1.13.0 containing this change is available at https://epr.elastic.co/search?package=atlassian_confluence |
Package atlassian_jira - 1.13.0 containing this change is available at https://epr.elastic.co/search?package=atlassian_jira |
Package bitdefender - 1.1.0 containing this change is available at https://epr.elastic.co/search?package=bitdefender |
Package box_events - 1.6.0 containing this change is available at https://epr.elastic.co/search?package=box_events |
Package cisco_secure_endpoint - 2.16.0 containing this change is available at https://epr.elastic.co/search?package=cisco_secure_endpoint |
Package cloudflare - 2.11.0 containing this change is available at https://epr.elastic.co/search?package=cloudflare |
Package forgerock - 1.4.0 containing this change is available at https://epr.elastic.co/search?package=forgerock |
Package github - 1.15.0 containing this change is available at https://epr.elastic.co/search?package=github |
Package jumpcloud - 1.1.0 containing this change is available at https://epr.elastic.co/search?package=jumpcloud |
Package m365_defender - 1.12.0 containing this change is available at https://epr.elastic.co/search?package=m365_defender |
Package microsoft_defender_endpoint - 2.14.0 containing this change is available at https://epr.elastic.co/search?package=microsoft_defender_endpoint |
Package microsoft_exchange_online_message_trace - 1.6.0 containing this change is available at https://epr.elastic.co/search?package=microsoft_exchange_online_message_trace |
Package mimecast - 1.12.0 containing this change is available at https://epr.elastic.co/search?package=mimecast |
Package okta - 1.23.0 containing this change is available at https://epr.elastic.co/search?package=okta |
Package panw_cortex_xdr - 1.13.0 containing this change is available at https://epr.elastic.co/search?package=panw_cortex_xdr |
Package slack - 1.7.0 containing this change is available at https://epr.elastic.co/search?package=slack |
Package ti_abusech - 1.14.0 containing this change is available at https://epr.elastic.co/search?package=ti_abusech |
Package ti_cif3 - 1.1.0 containing this change is available at https://epr.elastic.co/search?package=ti_cif3 |
Package ti_cybersixgill - 1.15.0 containing this change is available at https://epr.elastic.co/search?package=ti_cybersixgill |
Package ti_misp - 1.17.0 containing this change is available at https://epr.elastic.co/search?package=ti_misp |
Package ti_otx - 1.12.0 containing this change is available at https://epr.elastic.co/search?package=ti_otx |
Package ti_threatq - 1.13.0 containing this change is available at https://epr.elastic.co/search?package=ti_threatq |
Package tines - 1.1.0 containing this change is available at https://epr.elastic.co/search?package=tines |
Package zerofox - 1.13.0 containing this change is available at https://epr.elastic.co/search?package=zerofox |
Package zeronetworks - 1.1.0 containing this change is available at https://epr.elastic.co/search?package=zeronetworks |
Not all acceptable units are listed, but rather choose commonly used units that are already documented in other packages. akamai, atlassian_bitbucket, atlassian_confluence, atlassian_jira, bitdefender, box_events, cisco_secure_endpoint, cloudflare, cloudflare, forgerock, github, github, jumpcloud, m365_defender, microsoft_defender_endpoint, microsoft_exchange_online_message_trace, mimecast, okta, panw_cortex_xdr, slack, ti_abusech, ti_cif3, ti_cybersixgill, ti_cybersixgill, ti_misp, ti_otx, ti_threatq, tines, zerofox, zeronetworks
What does this PR do?
Add note in manifests of akamai, atlassian_bitbucket, atlassian_confluence, atlassian_jira, cisco_secure_endpoint, cloudflare, forgerock, github, m365_defender, microsoft_exchange_online_message_trace, okta, panw_cortex_xdr, slack, ti_cybersixgill, ti_misp, tines, zerofox and zeronetworks to show valid initial interval units where not already specified.
Not all acceptable units are listed, but rather choose commonly used units that are already documented in other packages.
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots