Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update embed youtube markup #1462

Merged
merged 2 commits into from
Mar 10, 2023
Merged

Update embed youtube markup #1462

merged 2 commits into from
Mar 10, 2023

Conversation

rafalp
Copy link
Owner

@rafalp rafalp commented Mar 10, 2023

Fixes #1450

Changes referrer policy to strict-origin-when-cross-origin which some YouTube videos require to play.

Also updates embed markup to use youtube-nocookie.com

@rafalp rafalp added the area: frontend This issue involves JavaScript, React.js and Node label Mar 10, 2023
@rafalp rafalp added this to the Misago 0.30 milestone Mar 10, 2023
@rafalp rafalp self-assigned this Mar 10, 2023
@rafalp rafalp added the area: backend This issue involves Python, Django or dependency (eg. database) label Mar 10, 2023
@rafalp rafalp merged commit e57b6b4 into master Mar 10, 2023
@rafalp rafalp deleted the update-youtube-embed branch March 10, 2023 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: backend This issue involves Python, Django or dependency (eg. database) area: frontend This issue involves JavaScript, React.js and Node
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some Youtube videos fail to embed
1 participant