Skip to content

fix: allow adding YouTube channels with bare URLs and unreliable validation#1067

Merged
koala73 merged 1 commit intomainfrom
fix/youtube-channel-bare-url
Mar 5, 2026
Merged

fix: allow adding YouTube channels with bare URLs and unreliable validation#1067
koala73 merged 1 commit intomainfrom
fix/youtube-channel-bare-url

Conversation

@koala73
Copy link
Owner

@koala73 koala73 commented Mar 5, 2026

Summary

  • Support bare YouTube channel URLs like youtube.com/AztecaNoticias (without @/c/channel prefix) in the custom channel input
  • Remove blocking channelExists === false validation — YouTube scraping from datacenter IPs frequently returns false negatives, preventing users from adding valid channels like @AztecaNoticias

Test plan

  • tsc --noEmit passes
  • All pre-push checks pass (57/57 tests)
  • Add https://www.youtube.com/AztecaNoticias via custom channel → should succeed
  • Add @AztecaNoticias via custom channel → should succeed

…dation

- Support bare YouTube URLs (youtube.com/ChannelName without @/c/channel prefix)
- Remove blocking channelExists check — YouTube scraping from datacenter IPs
  frequently returns false negatives, preventing users from adding valid channels
@vercel
Copy link

vercel bot commented Mar 5, 2026

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

Project Deployment Actions Updated (UTC)
worldmonitor Ready Ready Preview, Comment Mar 5, 2026 6:07pm

Request Review

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@koala73 koala73 merged commit 91499f0 into main Mar 5, 2026
3 checks passed
aldoyh pushed a commit to aldoyh/worldmonitor that referenced this pull request Mar 6, 2026
…dation (koala73#1067)

- Support bare YouTube URLs (youtube.com/ChannelName without @/c/channel prefix)
- Remove blocking channelExists check — YouTube scraping from datacenter IPs
  frequently returns false negatives, preventing users from adding valid channels
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.

1 participant