Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

Fix issue where ambient sensor data is never updated #90

Merged
merged 2 commits into from
Jul 2, 2018

Conversation

lukiffer
Copy link
Collaborator

@lukiffer lukiffer commented Jul 1, 2018

Oversight on my part originally - the get_ambient_sensor_data() method was never called when the ArloBaseStation.update() method is called.

@lukiffer lukiffer changed the title Fix issue where ambient sensor data is never updated WIP: Fix issue where ambient sensor data is never updated Jul 1, 2018
@coveralls
Copy link

coveralls commented Jul 1, 2018

Coverage Status

Coverage increased (+0.02%) to 78.819% when pulling f7a7e1e on lukiffer:fix/baby-stats-update into 0fccabe on tchellomello:master.

@lukiffer lukiffer changed the title WIP: Fix issue where ambient sensor data is never updated Fix issue where ambient sensor data is never updated Jul 1, 2018
@tchellomello tchellomello merged commit adae8d9 into tchellomello:master Jul 2, 2018
@tchellomello
Copy link
Owner

@lukiffer I'm going to publish the new package tonight. Can you submit a PR to home assistant to bump the pyarlo module too?

@lukiffer
Copy link
Collaborator Author

lukiffer commented Jul 2, 2018

@tchellomello I'm wrapping up the unit tests to get home-assistant/core#15074 merged. Should I just bump the package version with that PR or submit a separate one?

@tchellomello
Copy link
Owner

@lukiffer you can bump on that PR itself. Give me about 10 minutes and the new package will be published for you. I'll let you know.

@tchellomello tchellomello mentioned this pull request Jul 2, 2018
@tchellomello
Copy link
Owner

@lukiffer there we go!! https://pypi.org/project/pyarlo/0.1.9/ 0.1.9 is published!

Thanks @lukiffer

@lukiffer lukiffer deleted the fix/baby-stats-update branch July 2, 2018 07:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants