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

Zigbee2MQTT Devices need to actually be categorized #33

Merged
merged 2 commits into from
Feb 24, 2025

Conversation

ddellacosta
Copy link
Owner

@ddellacosta ddellacosta commented Feb 23, 2025

Re: #32

  • all devices that I can personally (manually) test should be identified correctly
  • there should be a test or tests exercising device decoding for all types included in the above point
  • get rid of as many PS warnings as possible, there are a bunch that have been hanging around a while

I started extending the device-specific tests to the UI but it is really still an ugly hack and more importantly it would take a bunch more work to be able to differentiate devices in the UI in a way I wouldn't just be refactoring later on, so I left that alone for now. Todo: fix up the device UI, obvs

…a handful of sensors and the window blinds closing device
@ddellacosta ddellacosta changed the title Issue #32 - Zigbee2MQTT Devices need to actually be categorized #32 - Zigbee2MQTT Devices need to actually be categorized Feb 23, 2025
@ddellacosta ddellacosta changed the title #32 - Zigbee2MQTT Devices need to actually be categorized Zigbee2MQTT Devices need to actually be categorized Feb 23, 2025
@ddellacosta ddellacosta linked an issue Feb 23, 2025 that may be closed by this pull request
…rsed. The rest will be lazily loaded

Also, got rid of a bunch of warnings
@ddellacosta ddellacosta marked this pull request as ready for review February 24, 2025 14:35
@ddellacosta ddellacosta merged commit fb48176 into main Feb 24, 2025
2 checks passed
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.

Zigbee2MQTT Devices need to actually be categorized
1 participant