Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for other fields #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

durera
Copy link

@durera durera commented Feb 26, 2017

I extended the properties supported by this python library based on the work in rainlake/node-lightify for my ibm-watson-iot/gateway-lightify project. Figured you may be interested in this (or a version of it) to add support for the additional properties.

  • fwVersion
  • groupId
  • online
  • alpha

Also, name has been made updateable, rather than fixed at initialization.

Mainly involved adding new getter methods and rewriting the binary data unpacking.

@andreasofthings
Copy link
Owner

Thanks for your pull request. Unfortunately, I've been busy recently, but see some light at the end of the tunnel in March. Let me review and merge then.

@andreasofthings
Copy link
Owner

Hi @durera ,

since this package is now unmaintained, you may consider submitting a pull request towards

https://github.com/tfriedel/python-lightify

@tfriedel will pick this up.

Regards,

Andreas

Repository owner deleted a comment from Soumya6Tiwari Feb 23, 2024
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