Download videos also?
greggmarshall opened this issue · comments
Now that the free learning is including videos, it would be really nice if the script could download the videos, especially since it appears the API is the only way to access free learning assets other than on-line.
Would this be hard to add?
Should be easy. Try installing script via pip install git+https://github.com/mjenczmyk/Packt-Publishing-Free-Learning.git@id --upgrade and add video file type to your config file, see if that works as expected.
You are right it was easy to add the video format and have it downloaded. Still running but I wonder what a .video file format actually is? A preliminary try on one file indicates it might be .zip.
That's right - it's a zip file (I've named it video file type to distinguish it from code files which are also zip packages). You can see it in line 201 in modified file. Does it work for you?
Like a champ. I can also confirm you have resolved the timeout issue completely. It has been running about 5 or 6 hours, it's picking up a number of videos I bought during one of the $5 sales....
Glad I could help ;) I've already merged it so you can install the script using the usual command pip install git+https://github.com/luk6xff/Packt-Publishing-Free-Learning.git@master --upgrade - there is also a pull request in case we decide we want the package on PyPi to simplify the command further.