search

w3c / wot-cg

Web of Things Community Group
https://www.w3.org/community/wot/
Other
16 stars 10 forks source link

4th tutorial video review #87

Closed handealptekin closed 8 months ago

handealptekin commented 10 months ago

https://murf.ai/share/lm6azmy3

egekorkan commented 9 months ago

I have more or less the same Ege's remarks just one bit: from minute 3:32 the music stops not sure if it is intended, but it would be better to keep it until the end.

Indeed. If possible, the audio should loop and that finishing sequence should come only at the end.

netlify[bot] commented 9 months ago

Deploy Preview for whatiswot failed.

Name Link
Latest commit 2d7cc5e5dfbf670a86033221dc37017ce3b500ce
Latest deploy log https://app.netlify.com/sites/whatiswot/deploys/653b77d81ec0740008b3f5ce
handealptekin commented 9 months ago

Video has been updated according to the requested changes. You can view it from the same link

egekorkan commented 9 months ago

We will keep it open since the music restarts abruptly at the moment.

egekorkan commented 8 months ago

@handealptekin will fix the music like in 3rd video and push the powerpoint together

handealptekin commented 8 months ago

I applied a fade-in/fade-out where the song ends and the other song begins again. It should be reviewed and decided upon

egekorkan commented 8 months ago

I think it is fine now, I have increased the fade amount as well. @handealptekin could you push the video file as well and then I can merge it.

handealptekin commented 8 months ago

I think it is fine now, I have increased the fade amount as well. @handealptekin could you push the video file as well and then I can merge it.

Should i push the video with the sound from murf.ai? And if so, should the subtitle be included in the video?

egekorkan commented 8 months ago

The video has been reviewed and can be merged. There will be a change to background music for all videos as mentioned at #114