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

UHF-11339: Added youtube no-cookie support #898

Closed
wants to merge 3 commits into from
Closed

Conversation

annadruid
Copy link
Contributor

@annadruid annadruid commented Feb 5, 2025

UHF-11339

What was done

  • Adds support for YouTube no-cookie urls

How to install

  • Make sure your instance is up and running on latest dev branch.
    • git pull origin dev
    • make fresh
  • Update the Helfi Platform config
    • composer require drupal/helfi_platform_config:dev-UHF-11339
  • Run make drush-updb drush-cr

How to test

Test in some instance that has video embeds, for example etusivu

  • Go edit some existing YouTube video media entity
  • Save the entity and go check the url of the video, it should have now no-cookie in the url
  • Add the video to some page to make sure it still works
  • Add some YouTube video with a normal url to a page, check that the url is a no-cookie url even though the media url is without no-cookie
  • Check that code follows our standards

Continuous documentation

  • This feature has been documented/the documentation has been updated
  • This change doesn't require updates to the documentation

Copy link

sonarqubecloud bot commented Feb 5, 2025

@annadruid annadruid closed this Feb 5, 2025
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