amdean / motion-notify

Notification system for Linux Motion providing upload to Google Drive and email notificaiton
GNU General Public License v3.0
32 stars 12 forks source link

Add Support to Save Uploads onto Daily Folders #8

Closed peteclarkez closed 9 years ago

peteclarkez commented 9 years ago

Hi I've been using this for a while but only recently updated to the OAuth version.

One the issues i always had was the number of files uploaded into a single folder made it impossible to browse back (or find a date of incident you were looking for)

I've updated the code so that it will create subfolders and save the files in there. There are issues with permissions, as the folder is owned by the service account who owns the p12 cert, but I've added tha ability to add more users as writers and readers.

Also reformatted the Readme for MD highlighting and spacing.

amdean commented 9 years ago

This looks great. I was having the same problem with millions of files in my Google Drive folder.

Thanks for committing this!

peteclarkez commented 9 years ago

Found an issue where it wasn't finding the parent folder properly and created extra ones. Got a fix in my branch, but want to test it out first before creating another pull request