Skip to content

📝 Restore embedded video on API integration guide#793

Open
MattPoblete wants to merge 2 commits intomainfrom
fix/defindex-video
Open

📝 Restore embedded video on API integration guide#793
MattPoblete wants to merge 2 commits intomainfrom
fix/defindex-video

Conversation

@MattPoblete
Copy link
Collaborator

Summary

  • Restore the embedded video on the API integration guide docs

Closes #790

Copilot AI review requested due to automatic review settings February 12, 2026 11:31
@vercel
Copy link

vercel bot commented Feb 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
defindex-dapp Ready Ready Preview, Comment Feb 12, 2026 11:36am
defindex-landing Ready Ready Preview, Comment Feb 12, 2026 11:36am
defindex-rust-docs Ready Ready Preview, Comment Feb 12, 2026 11:36am

Request Review

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Restores the embedded video block in the API integration guide so readers can watch the walkthrough directly in the docs (per Issue #790).

Changes:

  • Reintroduced a GitBook {% embed %} YouTube video embed near the top of the API integration guide.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Prefer learning by watching? Check out our integration walkthrough:

[Watch the integration walkthrough on YouTube](https://www.youtube.com/watch?v=gz6GU5kAUXY&t=145s)
[Watch the integration walkthrough on YouTube](https://www.youtube.com/watch?v=gz6GU5kAUXY&t=145s):
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The embed URL contains an HTML-escaped query separator (&). In GitBook {% embed %} tags this is treated as a literal string, so YouTube receives amp;start=145 instead of start=145 and the video won’t start at the intended timestamp. Use a plain & between query parameters (or remove the extra param entirely).

Suggested change
[Watch the integration walkthrough on YouTube](https://www.youtube.com/watch?v=gz6GU5kAUXY&t=145s):
{% embed url="https://www.youtube.com/embed/gz6GU5kAUXY?si=54lqva3t6lzKjvdH&start=145" %}

Copilot uses AI. Check for mistakes.
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.

[Docs] Add embebed video to docs

2 participants