andrew-codechimp / HA-Battery-Notes

A Home Assistant integration to provide battery notes of devices
MIT License
459 stars 85 forks source link

Shelly BLU Door/Window identification #1676

Closed chemelli74 closed 2 months ago

chemelli74 commented 2 months ago

System Health details

System Information

version core-2024.5.4
installation_type Home Assistant Container
dev false
hassio false
docker true
user root
virtualenv false
python_version 3.12.2
os_name Linux
os_version 4.4.302+
arch x86_64
timezone Europe/Rome
config_dir /config
Home Assistant Community Store GitHub API | ok -- | -- GitHub Content | ok GitHub Web | ok GitHub API Calls Remaining | 5000 Installed Version | 1.34.0 Stage | running Available Repositories | 1395 Downloaded Repositories | 22
Home Assistant Cloud logged_in | false -- | -- can_reach_cert_server | ok can_reach_cloud_auth | ok can_reach_cloud | ok
Dashboards dashboards | 2 -- | -- resources | 12 views | 14 mode | storage
Recorder oldest_recorder_run | 11 maggio 2024 alle ore 18:43 -- | -- current_recorder_run | 21 maggio 2024 alle ore 11:44 estimated_db_size | 988.02 MiB database_engine | sqlite database_version | 3.44.2
Spotify api_endpoint_reachable | ok -- | --

Checklist

Describe the issue

Didn't check the code for the identification, but seems that is based on a field that is not always present.

image

Reproduction steps

  1. Install integration by HACS
  2. Enable integration in configuration.yaml
  3. Check battery+ for Shelly BLUE Door/Window device

Debug logs

No response

Diagnostics dump

No response

andrew-codechimp commented 2 months ago

The discovery only checks on the Model & Make In this case "BTHome sensor" by "Shelly" I can't see that in the library so raise a new device request with the correct battery type please.

chemelli74 commented 2 months ago

With Bluetooth is not so easy, unfortunately. Often the vendor is not reported :-( as it all depends from the latest packet received that is influenced by the signal strength

andrew-codechimp commented 2 months ago

Oh, that sounds horrible. I've not got any bluetooth myself, zigbee2mqtt everything. That will have to be a manual add rather than a library discovery then.