Skip to content

Allows older_than parameter to history() method#69

Merged
tchellomello merged 1 commit into
masterfrom
older_than
Nov 14, 2017
Merged

Allows older_than parameter to history() method#69
tchellomello merged 1 commit into
masterfrom
older_than

Conversation

@tchellomello

@tchellomello tchellomello commented Nov 14, 2017

Copy link
Copy Markdown
Member
  • Added ability to pass the parameter 'older_than' to history() method
  • Added script which allows count the number of videos and download them
scripts/ringcli.py -u foo -p bar --count
---------------------------------
Ring CLI
---------------------------------
	Counting videos linked on your Ring account.
	This may take some time....

	Total videos: 384
	Ding triggered: 32
	Motion triggered: 340
	On-Demand triggered: 12

======================

---------------------------------
Ring CLI
---------------------------------
scripts/ringcli.py -u foo -p bar --count
	1/384 Downloading 2017-11-14_00.57.16+00.00_motion_answered_64880679462719.mp4
	2/384 Downloading 2017-11-13_21.32.23+00.00_motion_not_answered_64880151491.mp4
======================

scripts/ringcli.py --help
usage: ringcli.py [-h] [-u USERNAME] [-p PASSWORD] [--count] [--download-all]

Ring Doorbell

optional arguments:
  -h, --help            show this help message and exit
  -u USERNAME, --username USERNAME
                        username for Ring account
  -p PASSWORD, --password PASSWORD
                        username for Ring account
  --count               count the number of videos on your Ring account
  --download-all        download all videos on your Ring account

https://github.com/tchellomello/python-ring-doorbell

Fixes: #63

Many thanks to @troopermax for reporting the issue and suggesting the addition

@tchellomello
- Added ability to pass the paramenter 'older_than' to history() method
e9fa01a 
 - Added script which allows count the number of videos and download them
@tchellomello tchellomello added this to the v.0.1.7 milestone Nov 14, 2017
@tchellomello tchellomello self-assigned this Nov 14, 2017
@coveralls

coveralls commented Nov 14, 2017 •
edited
Loading

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.08%) to 66.75% when pulling e9fa01a on older_than into b36576f on master.

2 similar comments
@coveralls

coveralls commented Nov 14, 2017

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.08%) to 66.75% when pulling e9fa01a on older_than into b36576f on master.

@coveralls

coveralls commented Nov 14, 2017

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.08%) to 66.75% when pulling e9fa01a on older_than into b36576f on master.

@tchellomello tchellomello merged commit d6cd227 into master Nov 14, 2017
@tchellomello tchellomello deleted the older_than branch November 14, 2017 05:50
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@tchellomello tchellomello

Labels

enhancement feature-request

Projects

None yet

Milestone

v.0.1.7

Development

Successfully merging this pull request may close these issues.

Doorbell history does not return all events

2 participants

@tchellomello @coveralls