Skip to content

chore(kno-12198): fix callout to use type prop#1362

Merged
scotidodson merged 2 commits intomainfrom
sd-KNO-12198-fix-callout-type
Mar 23, 2026
Merged

chore(kno-12198): fix callout to use type prop#1362
scotidodson merged 2 commits intomainfrom
sd-KNO-12198-fix-callout-type

Conversation

@scotidodson
Copy link
Contributor

Summary

  • Fixes the channel conditions callout in content/integrations/overview.mdx to use type="info" instead of emoji="💡". Using a custom emoji without a type drops the background color.

Changes

  • content/integrations/overview.mdxemoji="💡"type="info" on channel conditions callout

@linear
Copy link

linear bot commented Mar 23, 2026

@vercel
Copy link

vercel bot commented Mar 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 23, 2026 10:19pm

Request Review

cursor[bot]
cursor bot approved these changes Mar 23, 2026
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.


<Callout
emoji="💡"
type="info"
Copy link

Choose a reason for hiding this comment

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

Callout type mismatch: info used for warning content

Low Severity

This Callout uses type="info" but its content warns about a side effect — sandbox mode silently prevents channel conditions from being evaluated, and tells users to disable sandbox mode for conditions to take effect. Per the project's Callout validation rules, content that warns about potential problems or side effects fits type="warning" rather than type="info".

Fix in Cursor Fix in Web

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this is appropriate as info. This callout describes expected behavior. We recently had a customer face confusion trying to use these things together, so this callout is an attempt to make the information more visible.

@scotidodson scotidodson merged commit 8dac9d5 into main Mar 23, 2026
5 checks passed
@scotidodson scotidodson deleted the sd-KNO-12198-fix-callout-type branch March 23, 2026 22:20
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.

1 participant