fix(website): use CDN video asset for intelligence demo - #991
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题背景
官网首页第二屏的视频 Tab 显示的是轻微缩放的本地占位素材,且上一版部署后官网首页持续返回 500。
证据与根因
Event handlers cannot be passed to Client Component props,原因是服务端OnlineHomePage内联<video onCanPlay>。https://cdn.shulex-voc.com/flatkey/model-examples/seedance-f1-wet-track.mp4。修改范围
IntelligenceVideo,在用户切换 Video tab 或素材加载完成后安全触发播放。风险与影响
仅影响官网 Next.js 首页媒体展示和客户端播放逻辑;不影响 Go API、console 或 router。视频依赖现有 CDN 资源。
验证
bun run typecheck通过。bun run lint通过(仅保留既有<img>warnings)。bun run build通过。Router deploy: not required.
Other deploy targets:
newapi-web/ staging website required; console/router not required.