Skip to content

showcase:Pangs Rally#74

Open
pangsdev wants to merge 6 commits into
Virtual-Protocol:mainfrom
pangsdev:main
Open

showcase:Pangs Rally#74
pangsdev wants to merge 6 commits into
Virtual-Protocol:mainfrom
pangsdev:main

Conversation

@pangsdev

@pangsdev pangsdev commented Jul 22, 2026

Copy link
Copy Markdown

Showcase Project

What shipped

Project package

  • Added or updated showcase/pangs-rally/showcase.json
  • Added demo artifacts, prompt, proof, or redacted report
  • Added reusable skill under showcase/<project-slug>/skills/<skill-name>/ when it belongs to this project package
  • Used top-level skills/<skill-name>/ only when the skill is shared across projects
  • Set skills[].sourcePath in showcase.json for any skill committed in this repo

@pangsdev pangsdev changed the title Create showcase.json showcase:Pangs Rally Jul 22, 2026

@LauJoeYing LauJoeYing left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the submission! Before this can be merged, the hero card needs some fixes in showcase.json:

  1. posterUrl is empty (""). there's no image asset in the PR for now. Could you add a poster/screenshot (include assets/poster.jpg) and point posterUrl at it?

  2. No videoUrl, the visual block has a videoLabel ("Watch the gameplay demo on X") but nothing to actually play. links.video points to an X/Twitter status page, which won't embed as a playable video. Could you either:

  • link directly to a hosted video file (e.g. an .mp4), or
  • clarify how this should render if a tweet link is all that's available?
    artifacts is empty consider adding at least a proof/demo link here so reviewers have something to click into.

@pangsdev

Copy link
Copy Markdown
Author

I've uploaded the hosted video (.mp4) and poster (.png) directly to the assets folder as suggested, and updated the showcase.json with the correct URLs and artifacts. Thanks for the guidance!

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