Skip to content

[docs] : crowdstrike logo as icon in docs#3624

Merged
akshaydeo merged 1 commit into
devfrom
05-20-_docs_crowdstrike_logo_as_icon_in_docs
May 20, 2026
Merged

[docs] : crowdstrike logo as icon in docs#3624
akshaydeo merged 1 commit into
devfrom
05-20-_docs_crowdstrike_logo_as_icon_in_docs

Conversation

@Madhuvod
Copy link
Copy Markdown
Contributor

Summary

Briefly explain the purpose of this PR and the problem it solves.

Changes

  • What was changed and why
  • Any notable design decisions or trade-offs

Type of change

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Chore/CI

Affected areas

  • Core (Go)
  • Transports (HTTP)
  • Providers/Integrations
  • Plugins
  • UI (React)
  • Docs

How to test

Describe the steps to validate this change. Include commands and expected outcomes.

# Core/Transports
go version
go test ./...

# UI
cd ui
pnpm i || npm i
pnpm test || npm test
pnpm build || npm run build

If adding new configs or environment variables, document them here.

Screenshots/Recordings

If UI changes, add before/after screenshots or short clips.

Breaking changes

  • Yes
  • No

If yes, describe impact and migration instructions.

Related issues

Link related issues and discussions. Example: Closes #123

Security considerations

Note any security implications (auth, secrets, PII, sandboxing, etc.).

Checklist

  • I read docs/contributing/README.md and followed the guidelines
  • I added/updated tests where appropriate
  • I updated documentation where needed
  • I verified builds succeed (Go and UI)
  • I verified the CI pipeline passes locally if applicable

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 20, 2026

Review Change Stack

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Updated the CrowdStrike AIDR Guardrails provider documentation with the official CrowdStrike brand icon. The icon has been replaced across enterprise and integration documentation pages to ensure consistent branding and visual identification. This enhancement improves the visual clarity and professional presentation of the CrowdStrike AIDR Guardrails provider throughout the documentation.

Walkthrough

The PR updates icon references in CrowdStrike AIDR guardrails documentation, replacing generic binoculars icons with brand-specific CrowdStrike logo assets across two documentation files.

Changes

CrowdStrike AIDR Icon Assets

Layer / File(s) Summary
Icon asset references for CrowdStrike AIDR documentation
docs/enterprise/guardrails.mdx, docs/integrations/guardrails/crowdstrike-aidr.mdx
Updates icon references in the enterprise guardrails provider cards and integration documentation frontmatter to point to new CrowdStrike logo assets (/media/crowdstrike-white.svg and /media/crowdstrike-white.png).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • maximhq/bifrost#3540: Both PRs modify the CrowdStrike AIDR guardrails documentation—this PR swaps the provider card icon assets while the other PR extends the provider documentation in the same files.

Suggested reviewers

  • akshaydeo
  • danpiths

Poem

🎯 A rabbit hops through docs so fine,
Trading binoculars for a brand new sign,
CrowdStrike's logo now shines bright,
In enterprise and integration's light! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (2 warnings, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is completely unfilled; it only contains the empty template without any actual content about the changes, reasoning, affected areas, or test steps. Fill in all template sections with specific details: summarize the icon change purpose, explain why CrowdStrike logo is better, mark Documentation as affected area, and confirm testing steps completed.
Linked Issues check ⚠️ Warning The linked issue #123 concerns Files API support for providers, which is completely unrelated to documentation icon updates for CrowdStrike. Remove the incorrect linked issue #123 or link to the correct issue that tracks documentation improvements and CrowdStrike integration updates.
Out of Scope Changes check ❓ Inconclusive The PR contains only icon/logo updates to CrowdStrike documentation, which is in scope. However, it is linked to an unrelated Files API issue, suggesting potential scope confusion. Clarify the relationship between this PR and linked issue #123; if unrelated, remove the link to avoid confusion about actual PR scope.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The PR title clearly describes the main change: replacing the binoculars icon with CrowdStrike logo in documentation files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 05-20-_docs_crowdstrike_logo_as_icon_in_docs

Comment @coderabbitai help to get the list of available commands and usage tips.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 20, 2026

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Madhuvod Madhuvod force-pushed the 05-20-_docs_crowdstrike_logo_as_icon_in_docs branch from d319f5a to 3253c93 Compare May 20, 2026 11:35
@Madhuvod Madhuvod force-pushed the 05-20-_docs_crowdstrike_logo_as_icon_in_docs branch from 3253c93 to 11f1bc5 Compare May 20, 2026 11:50
@Madhuvod Madhuvod marked this pull request as ready for review May 20, 2026 11:51
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 20, 2026

Confidence Score: 4/5

Safe to merge; the change is limited to doc assets and icon metadata with no impact on application logic.

The only concerns are visual consistency (SVG vs. PNG split across two files) and the black background in the SVG that could look jarring in light-mode docs. Neither affects functionality.

docs/media/crowdstrike-white.svg — the black background fill should be reviewed in light and dark themes before publishing.

Important Files Changed

Filename Overview
docs/enterprise/guardrails.mdx Card icon for CrowdStrike AIDR changed from Font Awesome binoculars to custom SVG path /media/crowdstrike-white.svg
docs/integrations/guardrails/crowdstrike-aidr.mdx Page-level front matter icon changed from binoculars to /media/crowdstrike-white.png — uses PNG while the card in guardrails.mdx uses the SVG variant
docs/media/crowdstrike-white.svg New SVG icon with hard-coded black background rectangle; the white-on-black logo may clash with light-mode doc themes
docs/media/crowdstrike-white.png New PNG version of the CrowdStrike logo added alongside the SVG

Reviews (1): Last reviewed commit: "[docs] : crowdstrike logo as icon in doc..." | Re-trigger Greptile

Comment thread docs/integrations/guardrails/crowdstrike-aidr.mdx
Comment thread docs/media/crowdstrike-white.svg
@coderabbitai coderabbitai Bot requested review from akshaydeo and danpiths May 20, 2026 11:54
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/integrations/guardrails/crowdstrike-aidr.mdx`:
- Line 4: Replace the PNG icon reference with the SVG in the CrowdStrike AIDR
integration metadata: update the icon field value "icon:
"/media/crowdstrike-white.png"" to use the SVG filename (the same value used in
the guardrails overview) so both documents reference the scalable SVG asset;
verify the corresponding "/media/crowdstrike-white.svg" file exists and update
any other occurrences of "crowdstrike-white.png" to "crowdstrike-white.svg" for
consistency.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 716bea8c-aa33-468b-ae6e-a776bf6d46ad

📥 Commits

Reviewing files that changed from the base of the PR and between f0b695b and 11f1bc5.

⛔ Files ignored due to path filters (2)
  • docs/media/crowdstrike-white.png is excluded by !**/*.png
  • docs/media/crowdstrike-white.svg is excluded by !**/*.svg
📒 Files selected for processing (2)
  • docs/enterprise/guardrails.mdx
  • docs/integrations/guardrails/crowdstrike-aidr.mdx

Comment thread docs/integrations/guardrails/crowdstrike-aidr.mdx
@akshaydeo akshaydeo merged commit bb0c74b into dev May 20, 2026
15 of 16 checks passed
@akshaydeo akshaydeo deleted the 05-20-_docs_crowdstrike_logo_as_icon_in_docs branch May 20, 2026 12:04
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.

Files API Support

3 participants