Use network_status instead of homescreen #119
Conversation
Use network_status to check if System is armed or disarmed
blinkpy/sync_module.py
Outdated
| @@ -114,6 +117,7 @@ def refresh(self, force_cache=False): | |||
| self.events = self.get_events() | |||
| self.videos = self.get_videos() | |||
| self.homescreen = api.request_homescreen(self.blink) | |||
| self.network_info = api.request_network_status(self.blink,self.network_id) | |||
There was a problem hiding this comment.
line too long (82 > 79 characters)
missing whitespace after ','
blinkpy/sync_module.py
Outdated
| @@ -88,6 +89,8 @@ def start(self): | |||
|
|
|||
| self.homescreen = api.request_homescreen(self.blink) | |||
|
|
|||
| self.network_info = api.request_network_status(self.blink,self.network_id) | |||
There was a problem hiding this comment.
line too long (82 > 79 characters)
missing whitespace after ','
blinkpy/sync_module.py
Outdated
| @@ -114,6 +118,8 @@ def refresh(self, force_cache=False): | |||
| self.events = self.get_events() | |||
| self.videos = self.get_videos() | |||
| self.homescreen = api.request_homescreen(self.blink) | |||
| self.network_info = api.request_network_status(self.blink, | |||
blinkpy/sync_module.py
Outdated
| @@ -88,6 +89,9 @@ def start(self): | |||
|
|
|||
| self.homescreen = api.request_homescreen(self.blink) | |||
|
|
|||
| self.network_info = api.request_network_status(self.blink, | |||
blinkpy/sync_module.py
Outdated
| @@ -114,6 +117,7 @@ def refresh(self, force_cache=False): | |||
| self.events = self.get_events() | |||
| self.videos = self.get_videos() | |||
| self.homescreen = api.request_homescreen(self.blink) | |||
| self.network_info = api.request_network_status(self.blink, self.network_id) | |||
There was a problem hiding this comment.
line too long (83 > 79 characters)
There was a problem hiding this comment.
To fix this:
| self.network_info = api.request_network_status(self.blink, self.network_id) | |
| self.network_info = api.request_network_status(self.blink, | |
| self.network_id) |
|
The |
Co-Authored-By: md-reddevil <16707502+md-reddevil@users.noreply.github.com>
|
The test fix I gave you was incorrect, I misread the code. I just went in and fixed it myself and cleaned up the linting error, so it's ready to merge now. Thanks for the continued contributions! |
Use network_status to check if System is armed or disarmed
Description:
Related issue (if applicable): fixes #
fixes #118
Checklist:
toxrun successfully PR cannot be meged unless tests pass