fix(docs): replace broken demo video links in identity and payments READMEs#106
Open
alelpoan wants to merge 2 commits into
Open
fix(docs): replace broken demo video links in identity and payments READMEs#106alelpoan wants to merge 2 commits into
alelpoan wants to merge 2 commits into
Conversation
WalkthroughThis PR updates two demo README files by replacing previous video references with embedded, centered YouTube player links. Both the identity and payments demo READMEs now display clickable YouTube thumbnail embeds in their respective "Demo Video" sections using HTML ChangesDemo Video Documentation Updates
🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Fix broken demo video links in identity and payments READMEs.
Both README files referenced
github.com/user-attachments/assets/...video links that return 404. Replaced with clickable YouTube preview thumbnails linking to the correct videos already used on the official docs site at agentcommercekit.com.Note: GitHub Markdown does not support
<iframe>embeds, so a clickable thumbnail image is the standard approach for linking to videos in READMEs.Changes:
demos/identity/README.md: broken
user-attachmentsasset link →YouTube thumbnail preview for
https://www.youtube.com/watch?v=CDJ8pm_KlOwdemos/payments/README.md: broken
user-attachmentsasset link →YouTube thumbnail preview for
https://www.youtube.com/watch?v=49VaG1-u1i0Test plan:
Documentation-only change; no code paths affected.
Summary by CodeRabbit
Closes #107
Screenshots
Before (broken 404 video):
After (clickable YouTube preview):
Screenshots
Before (broken 404 video):
After (clickable YouTube preview):