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.

Sign up for GitHub

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

Ajout du unique_id au sensor d'énergie #296

Merged
merged 1 commit into from
Oct 16, 2023
+2 −1
Merged

Ajout du unique_id au sensor d'énergie #296

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion custom_components/hilo/sensor.py
100644 → 100755
View file
Open in desktop
Copy link
Contributor

@maxyvon maxyvon Oct 16, 2023 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Est-ce qu'on serait pas mieux de mettre self._attr_name = f"Hilo Energy {device.name}" pour le nom de l'entité ?
J'aurais mis self._attr_unique_id = f"{slugify(device.name)}-energy" pour être comme les autres sensors mais ça va mettre en double tout les sensor d'energy de ceux qui ont deja fait la mise a jour donc je le changrais pas juste pour ça.
Penses-tu que c'est faisable de le lier avec le device? Dans la classe d'IntegrationSensor il y a la donnée device_info qui pourrait être utilisé. (Elle n'est présentement pas utilisé dans l'intégration)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça me parait être une bonne suggestion par contre comme tu dis ça causerait du dédoublage.

As-tu la doc pas loin pour l’integration sensor? C’est pas fou comme idée.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le changement de nom ne fera pas de dédoublage mais je crois que le unique_id oui puisque les entité n'auront plus le même id.

Par contre pour je crois qu'il n'aura pas de dédoublage si on lie l'entité avec le device tout en gardant le même unique_id qui vient d'être ajouté.

J'ai pas trouver grand chose pour l’integration sensor à part sa définition dans le github de home assistant.
https://github.com/home-assistant/core/blob/dev/homeassistant/components/integration/sensor.py

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J’apprécie tes commentaires constructifs. Je suis pas un programmeur pour 2 cennes alors ça m’aide pas mal à m’améliorer!

Original file line number Diff line number Diff line change
Expand Up @@ -243,7 +243,8 @@ class EnergySensor(IntegrationSensor):

def __init__(self, device):
self._device = device
self._attr_name = f"hilo_energy_{slugify(device.name)}"
self._attr_name = f"Hilo Energy {slugify(device.name)}"
self._attr_unique_id = f"hilo_energy_{slugify(device.name)}"
self._unit_of_measurement = ENERGY_WATT_HOUR
self._unit_prefix = None
if device.type == "Meter":
Expand Down