Skip to content

Adds demo video links to SentinelOne, Crowdstrike, Microsoft Defender XDR, and Microsoft Defender Endpoint #17889

Open
jmikell821 wants to merge 3 commits intomainfrom
docs-enhancement/add-videos-integrations
Open

Adds demo video links to SentinelOne, Crowdstrike, Microsoft Defender XDR, and Microsoft Defender Endpoint #17889
jmikell821 wants to merge 3 commits intomainfrom
docs-enhancement/add-videos-integrations

Conversation

@jmikell821
Copy link
Contributor

Proposed commit message

Adds demo video links to SentinelOne, Crowdstrike, Microsoft Defender XDR, and Microsoft Defender Endpoint.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

Ensure the videos appear as expected.

Related issues

Closes https://github.com/elastic/docs-content-internal/issues/369
Closes https://github.com/elastic/docs-content-internal/issues/370
Closes https://github.com/elastic/docs-content-internal/issues/368

@jmikell821 jmikell821 self-assigned this Mar 18, 2026
@jmikell821 jmikell821 requested a review from a team as a code owner March 18, 2026 22:06
@jmikell821 jmikell821 added the enhancement New feature or request label Mar 18, 2026
@github-actions
Copy link
Contributor

Please run these commands to update changelogs and build docs:

for pkg in crowdstrike m365_defender microsoft_defender_endpoint sentinel_one; do
  cd packages/$pkg
  elastic-package changelog add --type enhancement --description "Improve documentation" --link "https://github.com/elastic/integrations/pull/17889" --next minor
  elastic-package build
  cd ../..
done
git add -u
git commit -m "docs: update changelogs and build documentation"
git push

Prerequisite: go install github.com/elastic/elastic-package

@github-actions
Copy link
Contributor

github-actions bot commented Mar 18, 2026

Vale Linting Results

Summary: 2 suggestions found

💡 Suggestions (2)
File Line Rule Message
packages/sentinel_one/_dev/build/docs/README.md 5 Elastic.Semicolons Use semicolons judiciously.
packages/sentinel_one/docs/README.md 5 Elastic.Semicolons Use semicolons judiciously.

The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@theletterf
Copy link
Contributor

+CC @KOTungseth since we had a convo on videos in docs already.

@andrewkroh andrewkroh added Integration:crowdstrike CrowdStrike Integration:m365_defender Microsoft Defender XDR Integration:microsoft_defender_endpoint Microsoft Defender for Endpoint Integration:sentinel_one SentinelOne documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Mar 19, 2026
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @jmikell821

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

LGTM.
Suggestions for discussion and later consideration (not blocking):

  • Consider less formal wording for intros.
  • Add duration. Users are more likely to engage and watch the whole video when they know ahead of time how much time they are investing.

Something like " Check out this demo. (Duration:13:55)"

Copy link
Contributor

@alaudazzi alaudazzi left a comment

Choose a reason for hiding this comment

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

LGTM

# newer versions go on top
- version: "2.5.0"
changes:
- description: Improve documentation
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- description: Improve documentation
- description: Add demo video link to integration documentation.

# newer versions go on top
- version: "4.5.0"
changes:
- description: Improve documentation
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- description: Improve documentation
- description: Add demo video link to integration documentation.

# newer versions go on top
- version: "5.12.0"
changes:
- description: Improve documentation
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- description: Improve documentation
- description: Add demo video link to integration documentation.

# newer versions go on top
- version: "3.12.0"
changes:
- description: Improve documentation
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- description: Improve documentation
- description: Add demo video link to integration documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:crowdstrike CrowdStrike Integration:m365_defender Microsoft Defender XDR Integration:microsoft_defender_endpoint Microsoft Defender for Endpoint Integration:sentinel_one SentinelOne Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants