search

yuppity / unifi-video-api

Python API for UniFi Video
MIT License
59 stars 11 forks source link

add option to download motion image #17

Closed donaldrs closed 4 years ago

yuppity commented 4 years ago

Based on some quick testing, it seemed like the motion image was not available for recordings of type fullTimeRecording; I wrote in an early return to avoid sending the HTTP request. Please let me know if you think this is wrong.

donaldrs commented 4 years ago

No, I should have added that myself. Sorry, I missed that case, I only have motion recordings myself.

Thanks again for your great work! It made my own small project possible. I’ve used any programming languages in the past, but never Python (hence the wrong use of spacing), so I was glad this part was already implemented by you. And it works great!

Richard

From: yuppity notifications@github.com Reply to: yuppity/unifi-video-api reply@reply.github.com Date: Friday, 31 July 2020 at 16:58 To: yuppity/unifi-video-api unifi-video-api@noreply.github.com Cc: donaldrs reg@hierzo.com, Author author@noreply.github.com Subject: Re: [yuppity/unifi-video-api] add option to download motion image (#17)

Based on some quick testing, it seemed like the motion image was not available for recordings of type fullTimeRecording; I wrote in an early return to avoid sending the HTTP request. Please let me know if you think this is wrong.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.