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

LZW31 (Black Series) driver child device updates and cleanup #62

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

Conversation

csstup
Copy link
Contributor

@csstup csstup commented Jun 29, 2023

Fixes:

  • Updated child driver implementation.
  • Refactored child methods.
  • Added missing child methods for compatibility with HE child drivers.
  • Use child driver logging correctly.

Cleanup:

  • Removed references and logic supporting LED notifications. Non existant on Black series devices.
  • Removed unsupported parameters on Black series.
  • Removed unused commands from device UI
  • Updated parameter presentation on device page to use titles and descriptions separately
  • Removed Polling capability
  • Removed some ST styles from parameter presentation
  • Cleaned up some indentation

Fixes:
- Updated child driver implementation.
- Refactored child methods.
- Added missing child methods for compatibility with HE child drivers.   
- Use child driver logging correctly.

Cleanup:
- Removed references and logic supporting LED notifications.  Non existant on Black series devices.
- Removed unsupported parameters on Black series.
- Removed unused commands from device UI
- Updated parameter presentation on device page to use titles and descriptions separately
- Removed Polling capability
- Removed some ST styles from parameter presentation
- Cleaned up some indentation
@csstup
Copy link
Contributor Author

csstup commented Jun 29, 2023

BTW Im not updating the version dates or revision comments, leaving those up to you guys to summarize!

@InovelliUSA
Copy link
Owner

Had repo notifications off so didn't see this, but willing to check it out now. After months of testing you haven't experienced any issues with these updates?

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