search

decidim / documentation

Documentation site for Decidim project built with Antora
https://docs.decidim.org/
9 stars 23 forks source link

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

Open codePau opened 2 years ago

codePau commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

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