Ajout de config du module : Neo Coolcam PD07Z

[Anakins]

Description

Ajout de la configuration du module : Neo Coolcam PD07Z (Multisensor)

ID Fabricant : 600
ID Type : 16
ID Produit : 1824

Suggested changelog entry

Ajout de la configuration du module : Neo Coolcam PD07Z (Multisensor)

Related issues/external references

Fixes #

Types of changes

• Bug fix (non-breaking change which fixes)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
  • This change is only breaking for integrators, not for external standards or end-users.
• Documentation improvement

PR checklist

• I have checked there is no other PR open for the same change.
• I have read the Contribution Guidelines.
• I grant the project the right to include and distribute the code under the GNU.
• I have added tests to cover my changes.
• I have verified that the code complies with the projects coding standards.
• [Required for new sniffs] I have added MD documentation for the sniff.

[Salvialf]

Merci pour la PR, celle-ci est bien structurée et les IDs correspondent bien au NAS-PD07Z.

Un manque a priori : le PD07Z est un capteur 5-en-1 (PIR + température + humidité + luminosité + tamper), or la config ne déclare que Motion/Luminance/Tamper/Battery. Il manquerait donc :

"Temperature": {},
"Humidity": {}

Tu peux comparer avec nas-ds07z.json du même dossier qui les déclare. Comme sur la #58, une capture de la modale Valeurs de ton module permettra de confirmer ce qu'il remonte réellement.

Au passage, le couple "firmProperties": 1 + clé "default" ne sert que lorsqu'un module a des properties différentes selon la version du firmware. Ici il n'y a qu'un seul jeu : tu peux mettre les properties à plat et supprimer firmProperties, comme le font les autres configs du dossier (nas-pd01z.json, nas-ds07z.json). Idem sur la #58.

Deux détails : pour le champ name, utilise plutôt le modèle du produit (ex. NAS-PD07Z 5 in 1 Motion Sensor), et l'indentation du bloc properties est en tabs alors que le reste du fichier est en 4 espaces.

[Anakins]

C'et fait merci.