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

Delete one of the videos in https://docs.decidim.org/en/whitepaper/decidim-a-brief-overview/#_what_is_decidim #73

Open
codePau opened this issue Dec 14, 2021 · 4 comments

Comments

@codePau
Copy link
Contributor

codePau commented Dec 14, 2021

There is a video linked and another video embedded right below it. The second video does not add new information, so my proposal is to embed the first one and delete the second one.
Screenshot 2021-12-14 at 22 36 15

@andreslucena
Copy link
Member

Nice catch! I didn't see that there's a linked video and another embedded, that's weird for sure.

This is where it needs to be changed:
https://github.com/decidim/documentation/blame/95ec7a5a36b80a0670ae32e573307068a7788c63/en/modules/whitepaper/pages/what-is-decidim.adoc#L7

And these are the videos:

https://youtu.be/zhMMW0TENNA
https://youtu.be/f6JMgJAQ2tc

This section is mostly managed by @xabier - so what we should do with this one?

@ahukkanen
Copy link
Contributor

It's also a GDPR violation that the video is embedded on that page without user's prior consent.

Just a thought, how about making the "embedded" video an image which links to the YouTube page that already handles user's cookie consent properly.

@andreslucena
Copy link
Member

Just a thought, how about making the "embedded" video an image which links to the YouTube page that already handles user's cookie consent properly.

+1, this is a much better way for sure

@codePau
Copy link
Contributor Author

codePau commented Sep 17, 2022

Ok! Shall we leave the first one then? What do think @xabier ?

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

No branches or pull requests

3 participants