Skip to content

feat(telegram): point demo video at self-hosted walkthrough mp4 - #302

Merged
mashkovd merged 2 commits into
mainfrom
feat/telegram-demo-video-url
May 25, 2026
Merged

mashkovd merged 2 commits into
mainfrom
feat/telegram-demo-video-url

Conversation

@mashkovd

Copy link
Copy Markdown
Member

Summary

  • Switches DEMO_VIDEO_URL for the labs mctl-telegram service from the external YouTube link to the app's own route https://tg.mctl.ai/demo/walkthrough.mp4.
  • The /demo page will then render the walkthrough as a native <video> served by the service itself (no third-party host).
  • Hand-edited directly to avoid the known deploy-tooling footgun that strips colon-bearing env vars; do NOT round-trip this through any deploy CLI.

Merge ordering (important)

Merge / deploy this PR ONLY AFTER the mctl-telegram code PR (mctlhq/mctl-telegram#203) has shipped a release whose image includes the /demo/walkthrough.mp4 route. Until that image is live, this URL points at a 404.

Test plan

Switch DEMO_VIDEO_URL from the external YouTube link to the app's own
/demo/walkthrough.mp4 route so the /demo page serves the walkthrough
from the service itself. Hand-edited to avoid the deploy-CLI footgun
that strips colon-bearing env vars.
@claude

claude Bot commented May 25, 2026 •

Copy link
Copy Markdown

Claude finished @mashkovd's task in 26s —— View job


Trivial env var change — DEMO_VIDEO_URL switched from YouTube to the self-hosted path on the same domain (tg.mctl.ai) that already owns the ingress and PUBLIC_BASE_URL. YAML is valid, no secrets, no structural changes.

0 P1 / 0 P2 / 0 P3 findings. No P1/P2 findings, good to merge — but only after mctlhq/mctl-telegram#203 is released, as documented in the PR description.

@claude

claude Bot commented May 25, 2026 •

Copy link
Copy Markdown

Claude finished @mashkovd's task in 29s —— View job


Trivial values-only change. The ?v=1 cache-buster on DEMO_VIDEO_URL is a sensible fix for the Cloudflare stale-404 problem; the inline comment documents the rationale and the bump-when-replaced instruction clearly.

0 P1 / 0 P2 / 0 P3 findings. No P1/P2 findings, good to merge — but only after mctlhq/mctl-telegram#203 is released and the image serving /demo/walkthrough.mp4 is live, as documented in the PR description.

@mashkovd
mashkovd merged commit 92bb638 into main May 25, 2026
@mashkovd
mashkovd deleted the feat/telegram-demo-video-url branch May 25, 2026 00:54
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