lanec / zoom-batch-downloader

Download all your zoom cloud recordings
GNU General Public License v3.0
40 stars 20 forks source link

Problems with Path? #17

Closed mingomongo closed 1 year ago

mingomongo commented 1 year ago

Hi, I'm having issues with this code. It doesn't seem to like the path I'm inputting. In this screenshot I try two paths. The first error is for "G:\Shared drives\xxx xxx\Zoom Recordings\Downloads New Script", where it doesn't recognise it at all (this is a Google Drive virtual drive). The second error is for "C:\Users\xxxxxxxxs Studio 10\Documents\Test" which gets an error.

I have all the requirements and they're fully updated.

Batch downloader download path bug

AnessZurba commented 1 year ago

Hi, Thanks for your report. There was an issue in the template config file The variable should be called OUTPUT_PATH instead of PATH

I pushed an update fixing the template file. I also tried to rephrase the error message to convey the error more explicitly

The update includes a topic filter as well

Please let me know if you face any problems

mingomongo commented 1 year ago

This all works great, thanks a lot! A lot less hassle and better features than the one I used previously. Only thing I would add is a bit to the comment about users, they need to be separated by commas. Also I'm not sure if this script skips already downloaded videos?

AnessZurba commented 1 year ago

Glad to hear that, I'll edit the template file The script should skip already downloaded files, is it not working for you?

mingomongo commented 1 year ago

It does work actually now I'm using it, just wasn't sure as I hadn't seen it written anywhere that it was a feature