chore: add video for copilot org token#14909
Conversation
There was a problem hiding this comment.
Pull request overview
Adds an embedded walkthrough video to the Tokens reference docs showing how to mint GITHUB_COPILOT_TOKEN for org-owned repositories.
Changes:
- Inserts an HTML
<video>embed (with caption) under the org-owned repository PAT instructions. - Adds additional spacing around the “Setup” section.
Comments suppressed due to low confidence (2)
docs/src/content/docs/reference/tokens.md:351
- The
<video>element is missing the accessibility/performance attributes used elsewhere in the docs (e.g.,preload="metadata"and anaria-labelderived from the caption). Also consider using a richer fallback (e.g., a paragraph with a direct download link) so non-video-capable browsers still have a usable path.
<video
controls
loop
muted
playsinline
poster="/gh-aw/videos/create-pat-org-copilot.png"
style="width: 100%; aspect-ratio: 16/9;"
>
<source src="/gh-aw/videos/create-pat-org-copilot.mp4" type="video/mp4">
Your browser does not support the video tag.
</video>
docs/src/content/docs/reference/tokens.md:347
- The poster path points to
/gh-aw/videos/create-pat-org-copilot.png, but there doesn’t appear to be a correspondingdocs/public/videos/create-pat-org-copilot.pngasset. Either add the poster image or adjust/remove theposterattribute to avoid a broken request.
poster="/gh-aw/videos/create-pat-org-copilot.png"
style="width: 100%; aspect-ratio: 16/9;"
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
|
|
There was a problem hiding this comment.
There are two extra blank lines here that add unnecessary vertical space before the Setup section. Please remove the redundant empty lines to keep the docs formatting consistent.
| <div class="gh-aw-video-wrapper"> | ||
| <video | ||
| controls | ||
| loop | ||
| muted | ||
| playsinline | ||
| poster="/gh-aw/videos/create-pat-org-copilot.png" | ||
| style="width: 100%; aspect-ratio: 16/9;" |
There was a problem hiding this comment.
This video block isn’t using the existing video styling structure/classes (e.g., gh-aw-video-container + gh-aw-video-element) and instead relies on inline styles. Consider matching the structure used by docs/src/components/Video.astro so you get consistent border radius, aspect-ratio handling, and theming without inline CSS.
This issue also appears in the following locations of the same file:
- line 341
- line 346
| <div class="gh-aw-video-wrapper"> | |
| <video | |
| controls | |
| loop | |
| muted | |
| playsinline | |
| poster="/gh-aw/videos/create-pat-org-copilot.png" | |
| style="width: 100%; aspect-ratio: 16/9;" | |
| <div class="gh-aw-video-container"> | |
| <video | |
| class="gh-aw-video-element" | |
| controls | |
| loop | |
| muted | |
| playsinline | |
| poster="/gh-aw/videos/create-pat-org-copilot.png" |
COPILOT_GITHUB_TOKENfor a repository owned by an organization.