Skip to content

docs(kilo-docs): add setup videos to Slack, Telegram, and Google pages#8071

Merged
alexkgold merged 4 commits intomainfrom
docs/add-setup-videos
Apr 2, 2026
Merged

docs(kilo-docs): add setup videos to Slack, Telegram, and Google pages#8071
alexkgold merged 4 commits intomainfrom
docs/add-setup-videos

Conversation

@LigiaZ
Copy link
Copy Markdown
Contributor

@LigiaZ LigiaZ commented Apr 1, 2026

Summary

  • Adds a setup video callout to the Slack integration page
  • Adds a setup video callout to the Telegram integration page
  • Adds a setup video callout to the Google development tools page

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 1, 2026

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

Project Deployment Actions Updated (UTC)
docs (docs-staging) Ready Ready Preview, Comment Apr 2, 2026 7:58pm

Request Review

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot bot commented Apr 1, 2026

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Fix these issues in Kilo Cloud

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-docs/components/VideoEmbed.tsx 62 New video tag still renders an iframe embed, so it does not resolve the screenshare/preview problem called out in the PR discussion.
Other Observations (not in diff)

None.

Files Reviewed (7 files)
  • packages/kilo-docs/components/VideoEmbed.tsx - 1 issue
  • packages/kilo-docs/components/index.js
  • packages/kilo-docs/markdoc/tags/index.ts
  • packages/kilo-docs/markdoc/tags/video.markdoc.ts
  • packages/kilo-docs/pages/kiloclaw/chat-platforms/slack.md
  • packages/kilo-docs/pages/kiloclaw/chat-platforms/telegram.md
  • packages/kilo-docs/pages/kiloclaw/development-tools/google.md

Reviewed by gpt-5.4-2026-03-05 · 541,940 tokens

@LigiaZ
Copy link
Copy Markdown
Contributor Author

LigiaZ commented Apr 1, 2026

@alexkgold, a review, please? Can't add an iframe as this is not supported on screenshare.

@alexkgold
Copy link
Copy Markdown
Contributor

I don't think this worked.

image

borderRadius: "0.5rem",
}}
>
<iframe
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

WARNING: This still uses an iframe embed

The PR discussion notes that screenshare does not support iframe-based videos. VideoEmbed keeps the same <iframe> approach as the existing YouTube component, so switching pages to this new tag would still fail in the same way instead of fixing the preview issue.

@alexkgold alexkgold enabled auto-merge April 2, 2026 19:56
Copy link
Copy Markdown
Contributor

@alexkgold alexkgold left a comment

Choose a reason for hiding this comment

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

🎉

@alexkgold alexkgold merged commit 3b79453 into main Apr 2, 2026
15 checks passed
@alexkgold alexkgold deleted the docs/add-setup-videos branch April 2, 2026 19:59
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.

2 participants