Skip to content

showcase(clone-frame): align topics with common filter tags - #105

Merged
LauJoeYing merged 2 commits into
Virtual-Protocol:mainfrom
devclone20:showcase/clone-frame-topics
Aug 6, 2026
Merged

showcase(clone-frame): align topics with common filter tags#105
LauJoeYing merged 2 commits into
Virtual-Protocol:mainfrom
devclone20:showcase/clone-frame-topics

Conversation

@devclone20

Copy link
Copy Markdown

Applies the inline suggestion from the automated review on #96: leads the topics list with the common filter tags (agents, skills, developer-tools) so the card surfaces in the site filters, keeping desktop and byok as project-specific tags. No other changes.

Applies the inline suggestion from the automated review on PR 96:
lead with the common filter tags (agents, skills, developer-tools)
so the card surfaces in the site filters, keeping desktop and byok
as project-specific tags.
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

Thanks for this, looks like a good cleanup to align the topics with the site filters. Everything checks out on my end.


Refreshed after a new push. Any inline suggestions from the first pass may now be stale.

…oLabel

Applies the PR 105 review: visual.videoUrl (inline player) is the one
video pattern; the external-page fields links.video and visual.videoLabel
are removed. The video stays reachable from the artifacts list.
@devclone20

Copy link
Copy Markdown
Author

Done — removed links.video and visual.videoLabel; visual.videoUrl (the inline player) is now the single video pattern, and the video stays reachable from artifacts. node scripts/validate-showcase.mjs passes (56 manifests).

@LauJoeYing
LauJoeYing merged commit 754e170 into Virtual-Protocol:main Aug 6, 2026
2 checks passed
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.

3 participants