Skip to content

Add a convenience URL for our YouTube channel#184

Merged
swansama merged 1 commit into
mainfrom
push-prttxorsuvtn
May 27, 2026
Merged

Add a convenience URL for our YouTube channel#184
swansama merged 1 commit into
mainfrom
push-prttxorsuvtn

Conversation

@nathenharvey
Copy link
Copy Markdown
Contributor

@nathenharvey nathenharvey commented May 15, 2026

https://dora.community/youtube will redirect.

This also adds testing infrastructure for the change.

Fixes #183

Preview URL: https://deploy-preview-184--dora-community.netlify.app/youtube

@netlify
Copy link
Copy Markdown

netlify Bot commented May 15, 2026

Deploy Preview for dora-community ready!

Name Link
🔨 Latest commit e860d7c
🔍 Latest deploy log https://app.netlify.com/projects/dora-community/deploys/6a07515f00d7f40008c13f49
😎 Deploy Preview https://deploy-preview-184--dora-community.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nathenharvey nathenharvey marked this pull request as ready for review May 15, 2026 16:34
@nathenharvey nathenharvey requested a review from swansama May 15, 2026 16:34
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a system for managing external redirects, including a configuration file, a new ExternalRedirect component, and automated Playwright tests. Review feedback suggests moving the ExternalRedirect component to a dedicated file for better organization. There is also a recommendation to update the redirection logic to preserve query parameters, ensuring consistency and supporting analytics tracking while avoiding specific restricted parameters.

Comment thread src/routes/Routes.tsx Outdated
Comment thread src/routes/Routes.tsx Outdated
https://dora.community/youtube will redirect.

This also adds testing infrastructure for the change.

Fixes #183
Copy link
Copy Markdown
Contributor

@swansama swansama left a comment

Choose a reason for hiding this comment

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

lgtm

@swansama swansama merged commit df64d49 into main May 27, 2026
4 checks passed
@swansama swansama deleted the push-prttxorsuvtn branch May 27, 2026 14:52
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.

Add a convenience link for our youtube channel

2 participants