Skip to content

Replace the narrated tour with the version 3 walkthrough - #330

Merged
bifrost0x merged 1 commit into
mainfrom
docs/site-tour-video-3
Nov 11, 2026
Merged

bifrost0x merged 1 commit into
mainfrom
docs/site-tour-video-3

Conversation

@bifrost0x

Copy link
Copy Markdown
Owner

Problem and outcome

The demo page embedded the narrated walkthrough of version 2 inside a collapsed "Historical version 2 narrated tour" block. A narrated tour of version 3 is now published, so the page should show that one as the current tour.

  • site/video.js embeds the new video; every "Watch on YouTube" and chapter link on site/demo.html points to it.
  • The collapsed historical block becomes an open "Narrated tour" section.
  • Chapters and transcript are rewritten for the new video (eight chapters, 2:17).
  • Captions and poster are replaced under their existing file names.
  • The music credit for the old soundtrack is removed together with its styles. The new video does not use that track.
  • docs/webssh-3.md no longer describes the narrated video as historical.

The opt-in loading, the consent notice, the Content Security Policy and the privacy notice are unchanged.

Scope and links

Related issue/proposal: none, site content only.
Target milestone (if accepted into a release): none.

Validation

  • Exact tested revision and checks/results: this commit. pytest tests/test_documentation_surface.py tests/test_public_product_surface.py tests/test_branding.py tests/test_repository_hygiene.py tests/test_license_consistency.py tests/test_ci_change_scope.py: 190 passed. scripts/check_repository_hygiene.py and scripts/check_license_consistency.py pass. The full suite was not run locally.
  • Regression/acceptance coverage: test_demo_is_opt_in_and_has_local_preview_and_text_alternatives now expects the new video link, eight chapter times, the 2:17 caption and no "Historical" label.
  • Compatibility, upgrade and security impact: none for the application. Site only; no new third-party hosts, and nothing loads from YouTube before the visitor opts in.
  • Remaining deployment-specific checks or explicit limitations: the unbuilt page was loaded from a local static server (eight chapters, no link to the old video). The built and published site should be checked once after deployment, including loading the player.

Delivery

Merging implements the change; it does not publish a versioned release.

The product site embedded the version 2 walkthrough and labeled it as
historical material. A narrated tour of version 3 is now published, so
the demo page presents it as the current tour: new video, captions,
poster, chapters and transcript.

The music credit for the old soundtrack is removed; the new video does
not use that track.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-11T12:11:45.614319Z 91acd97 PR opened
🔒 Security Review ✅ Completed 2026-11-11T12:13:40.063286Z 91acd97 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@bifrost0x
bifrost0x merged commit 7fcf9c1 into main Nov 11, 2026
34 checks passed
@bifrost0x
bifrost0x deleted the docs/site-tour-video-3 branch November 11, 2026 12:18
@github-project-automation github-project-automation Bot moved this from Backlog to Done in WebSSH Roadmap Nov 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant