Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jul 18, 2025

The "Making an Offer" documentation page contained a broken YouTube video embed that was displaying "This video is private" to users, as shown in the issue screenshot. This created a poor user experience where visitors encountered a non-functional video player instead of useful content.

Problem

The video with ID Wy13GLmujhA embedded in /main/guides/getting-started/explainer-how-to-make-an-offer.md was inaccessible, showing "This video is private" even to users with Agoric accounts.

Solution

Removed the entire "Video Walkthrough" section containing the broken video embed. The page now flows cleanly from the tutorial introduction to the "How it Looks" section with the screenshot, providing a better user experience without the broken video interruption.

Changes

  • Removed 8 lines containing the broken video embed and surrounding tip block
  • No other content was affected - the page retains all essential information
  • Page structure remains intact with proper heading hierarchy

Testing

  • ✅ Link checker passes
  • ✅ No references to the broken video ID remain in the repository
  • ✅ Page content is complete and well-formatted
  • ✅ Verified minimal surgical change with no unintended modifications

The fix ensures users can access the documentation without encountering broken video content, improving the overall documentation experience.

Fixes #1276.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • www.youtube.com
    • Triggering command: `python3 -c
      import requests

video_ids = [
'Wy13GLmujhA', # The reported broken one
'T6h6TMuVHKQ',
'e9dMkC2oFh8',
'52SgGFpWjsY',
'YcWXqHPui_w',
'wW9-KuezPp8',
'38oTyVv_D9I',
'huTI9_Gq2ow',
'Wtq6dwsRdOQ',
'qHa7u8r62JQ',
'pWZUHJqj_Lo',
'WJ1MlHudpuM',
'XeHBMO7SckU'
]

for vid_id in video_ids:
try:
response = requests.get(f'REDACTED{vid_id}', timeout=10)
if 'Video unavailable' in response.text or 'This video is private' in response.text:
print(f'❌ {vid_id} - Video unavailable or private')
elif response.status_code == 200:
print(f'✅ {vid_id} - Video accessible')
else:
print(f'❓ {vid_id} - Status code: {response.status_code}')
except Exception as e:
print(f'❓ {vid_id} - Error: {e}')` (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 18, 2025

Deploying documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: aa7ae55
Status: ✅  Deploy successful!
Preview URL: https://563e8750.documentation-7tp.pages.dev
Branch Preview URL: https://copilot-fix-1276.documentation-7tp.pages.dev

View logs

@toliaqat
Copy link
Contributor

@copilot , some videos listed in our documentation have gone private. Let verify all youtube videos links and remove those embedded videos if they are not available on youtube now.

Copilot AI changed the title [WIP] Broken video link (or links?) Fix broken video link by removing private YouTube video Jul 18, 2025
Copilot AI requested a review from toliaqat July 18, 2025 05:57
Copilot finished work on behalf of toliaqat July 18, 2025 05:57
Copy link
Contributor Author

Copilot AI commented Jul 18, 2025

@copilot , some videos listed in our documentation have gone private. Let verify all youtube videos links and remove those embedded videos if they are not available on youtube now.

I found 16 unique YouTube video IDs across the documentation. However, I'm unable to directly verify their availability due to firewall restrictions that block access to YouTube (as shown in the original PR description).

The video IDs I found are:

  • 38oTyVv_D9I, 52SgGFpWjsY, QC5wQOcWynU, T6h6TMuVHKQ, WJ1MlHudpuM
  • Wtq6dwsRdOQ, XeHBMO7SckU, YcWXqHPui_w, e9dMkC2oFh8, faxrecQgEio
  • huTI9_Gq2ow, iyuo0ymTt4g, m5Pf2d1tHCs, pWZUHJqj_Lo, qHa7u8r62JQ, wW9-KuezPp8

Could you please provide a list of which specific video IDs are private/unavailable so I can remove them from the documentation? Alternatively, I can create a script to systematically check and remove videos if the firewall restrictions can be lifted.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • www.youtube.com
    • Triggering command: python3 /tmp/check_youtube_videos.py (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot finished work on behalf of toliaqat July 18, 2025 06:01
Copy link
Contributor

@amessbee amessbee left a comment

Choose a reason for hiding this comment

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

I checked with marketing and they don't have access to this video either. My guess is that this was created by Kris B. in his private account and is not public anymore.

It makes sense to remove it. The changes look good to me.

CC: @toliaqat

@toliaqat toliaqat marked this pull request as ready for review August 12, 2025 16:03
@toliaqat toliaqat merged commit f032de0 into main Aug 12, 2025
6 of 8 checks passed
@toliaqat toliaqat deleted the copilot/fix-1276 branch August 12, 2025 16:03
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.

Broken video link (or links?)

3 participants