Skip to content

Features needed for Home Assistant integration#4

Open
joncar wants to merge 5 commits into
rado0x54:developfrom
joncar:user/joncar/improvements
Open

Features needed for Home Assistant integration#4
joncar wants to merge 5 commits into
rado0x54:developfrom
joncar:user/joncar/improvements

Conversation

@joncar

@joncar joncar commented Oct 17, 2021

Copy link
Copy Markdown

I added several features I needed to use this in a Home Assistant integration:

  • Added support for parsing some new enums and fields.
  • Made it resilient to the addition of new event types since I kept finding more throughout the time I had a Snoo and new ones would throw an exception, breaking the long running connection.
  • Made the PubNub subscription resilient to authentication credentials expiring so it could remain connected for more than a few hours. NOTE: This is a breaking change to the constructor parameters.
  • Added a connection state callback.
  • Added a verbose option to the CLI tool to view the raw JSON responses to the API calls.

@rado0x54

Copy link
Copy Markdown
Owner

Phew, shame on me for not getting to this earlier. I will try to reactive this project soon and merge you contributions back to this project. Thanks for contributing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants