Update setup and ConfigEntry #1
Conversation
…ections in device info
…ns that do not have by-id mechanism
| # Validate input | ||
| await self.async_set_unique_id(DOMAIN + "_" + user_input[SETUP_SERIAL]) | ||
| await self.async_set_unique_id(DOMAIN + "_" + user_input[SETUP_SERIAL]) # TODO: switch to meter s/n |
There was a problem hiding this comment.
Ici le unique id concerne la config et non le device : cette configuration target bien sur le device (USB) et non sur le linky derrière. Je garderais SETUP_SERIAL comme identifiant de configuration je pense (d'ailleurs changer un uniq id dans HA entraine systématiquement un nouvel objet et une non continuité par rapport au précédents, sur des sensors cela veut dire un nouveau sensor).
Gardons l'actuel et enlevons ce TODO, cela évitera de mauvaises surprises à l'avenir.
| await asyncio.sleep(1) | ||
| return serial.serial_number | ||
|
|
||
| s_n = await asyncio.wait_for(read_serial_number(serial_reader), timeout=5) |
There was a problem hiding this comment.
Pourquoi ne pas faire de la fonction read_serial_number() une méthode de l'objet serial_reader ?
Vous pouvez d'ailleurs passer le timeout à 9, Home Assistant ne commence à afficher des warning d'initialisation qu'à partir de 10s.
| @@ -36,7 +39,7 @@ | |||
| MODE_HISTORIC_FIELD_SEPARATOR = b"\x20" | |||
|
|
|||
| LINE_END = b"\r\n" | |||
| FRAME_END = b"\r\x03\x02\n" | |||
|
La fatigue me guette... Je pense être sur une PR du projet principal :D |
|
J'ai eu peur en voyant la notification 😅 |
Changes: