Skip to content

add rendered product promo sample#35

Open
FahadBinHussain wants to merge 1 commit into
nexu-io:mainfrom
FahadBinHussain:feature/add-rendered-product-promo-sample
Open

add rendered product promo sample#35
FahadBinHussain wants to merge 1 commit into
nexu-io:mainfrom
FahadBinHussain:feature/add-rendered-product-promo-sample

Conversation

@FahadBinHussain

@FahadBinHussain FahadBinHussain commented Jun 8, 2026

Copy link
Copy Markdown

adds a real rendered mp4 sample for frame-product-promo and links it from both readmes.

what changed:

  • added docs/assets/renders/frame-product-promo.mp4
  • documented the project-render command shape in the showcase section
  • allowed only docs/assets/renders/*.mp4 through the generated-video gitignore rule

checks:

  • pnpm build
  • rendered with node packages/cli/dist/bin.js project-render proj_d2e348af-c4d --output docs/assets/renders/frame-product-promo.mp4 --stream-progress
  • ffprobe shows 14.366667s, 1920x1080, 60fps, 450622 bytes

closes #30

@lefarcen lefarcen requested a review from Siri-Ray June 8, 2026 07:02
@lefarcen lefarcen added size/XS Size XS (<20 LOC) risk/low Low risk type/docs Documentation labels Jun 8, 2026

@lefarcen lefarcen left a comment

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.

Hey @FahadBinHussain! 👋 Fast turnaround — you opened this the same morning you claimed it.

The .gitignore negation pattern is exactly right: renders/ catches every other renders directory while the two !docs/assets/renders/ lines carve out the specific asset path. Both READMEs get the same update (English + zh-CN in sync), and the command block uses a clear <project_id> placeholder so readers know where to substitute their own value. The ffprobe metadata in the PR description (1920×1080, 60fps, 14.4s) is a nice verification trail too.

This looks ready from a code standpoint — handing off to a maintainer to merge. ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk/low Low risk size/XS Size XS (<20 LOC) type/docs Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add a real rendered video example

2 participants