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

Améliorations Energy (Refresh, Commandes infos,) & Camera NOC (Commandes, Webhook) #18

Merged
merged 16 commits into from
Apr 6, 2024

Conversation

ThePiloo
Copy link
Contributor

@ThePiloo ThePiloo commented Apr 1, 2024

Description

Diverses petites améliorations.
Energie : Récupération de l'état de chauffe, Transfert du refresh vers la class principale Netatmo (dans l'optique de fusionner à terme avec les données Securité vu que l'API est désormais commune sur HomeStatus et HomesData)
Camera Extérieure : Gestion de nouveaux webhook

Suggested changelog entry

Améliorations

Types of changes

  • New feature (non-breaking change which adds functionality)

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.

Bascule du Refresh vers la class Netatmo
Reprise du refresh Energy (qui est commun à Security dans le nouveau modèle). L'adaptation de la class Security reste à réaliser
Ajout de la Gestion de nouveaux événements depuis l'inscription webhook (connection, disconnection, boot)
Mise à jour des commandes disponibles et récupérables
Mise à jour des commandes (principalement info)
Mise à jour des commandes (info)
Mise à jour des commandes info
@zoic21
Copy link
Contributor

zoic21 commented Apr 3, 2024

Bonjour,
Ca me semble bien sauf pour l'unification du refresh, c'est commun entre security et thermostat, pour moi c'est netatmo qui mélange tout mais je pense un jour ils vont corriger.

Suppression du traitement en erreur côté plugin des réponses Netatmo qui contiennent des erreurs concernant des modules ou des actions.
@ThePiloo
Copy link
Contributor Author

ThePiloo commented Apr 4, 2024

Bonjour, Ca me semble bien sauf pour l'unification du refresh, c'est commun entre security et thermostat, pour moi c'est netatmo qui mélange tout mais je pense un jour ils vont corriger.

C'est bien "volontaire" mais pas finalisé ... Ils mettent du temps (des années) mais ils ont opérés un premier rapprochement Home+Control & Home+Securité en utilisant "volontairement" des EndPoints communs
-------------> /homesdata: user's homes and their topology
-------------> /homestatus: current status of a home and the associated devices
Il y a aussi un troisième EndPoint qui est commun "/setstate" - Je ne crois pas qu'il soit utilisé par le plugin.

Je vais essayer d'avoir la confirmation écrite sur ce point

Ainsi le EndPoint "/gethomedata" encore utilisé par le plugin pour la partie Sécurité (mais déprécié) et les informations seront bien à récupérer en croissant les infos Homesdata et Homestatus

@ThePiloo
Copy link
Contributor Author

ThePiloo commented Apr 4, 2024

J'ai fait une correction sur la gestion des erreurs qui sont remontés avec un status "OK" ; Il faudra que je travaille le sujet car à date cela peut être un module débranché, ou une vraie erreur / anomalie , etc...

@ThePiloo
Copy link
Contributor Author

ThePiloo commented Apr 4, 2024

Réponse de Netatmo

_En effet nous avons décidé de mutualiser au maximum les endpoints entre les différents produits afin de faciliter d'une part la récupération d'informations pour les dévelopeurs, et aussi réduire potentiellement le nombre de calls à traiter de notre côté. Exemple : un /homesdata va retourner en 1 fois l'ensemble des produits Legrand/Bticino/Netatmo présents dans l'installation Ensuite, si le développeur veut par exemple uniquement récupérer les informations des caméras extérieur (NOC/-S) il peut demander dans sa requête /authorize seulement les scopes voulus (read_presence dans ce cas). Les autres produits de l'installation n'apparaîtront donc pas dans le retour JSON | En effet nous avons décidé de mutualiser au maximum les endpoints entre les différents produits afin de faciliter d'une part la récupération d'informations pour les dévelopeurs, et aussi réduire potentiellement le nombre de calls à traiter de notre côté. Exemple : un /homesdata va retourner en 1 fois l'ensemble des produits Legrand/Bticino/Netatmo présents dans l'installation Ensuite, si le développeur veut par exemple uniquement récupérer les informations des caméras extérieur (NOC/-S) il peut demander dans sa requête /authorize seulement les scopes voulus (read_presence dans ce cas). Les autres produits de l'installation n'apparaîtront donc pas dans le retour JSON

En effet nous avons décidé de mutualiser au maximum les endpoints entre les différents produits afin de faciliter d'une part la récupération d'informations pour les dévelopeurs, et aussi réduire potentiellement le nombre de calls à traiter de notre côté. Exemple : un /homesdata va retourner en 1 fois l'ensemble des produits Legrand/Bticino/Netatmo présents dans l'installation Ensuite, si le développeur veut par exemple uniquement récupérer les informations des caméras extérieur (NOC/-S) il peut demander dans sa requête /authorize seulement les scopes voulus (read_presence dans ce cas). Les autres produits de l'installation n'apparaîtront donc pas dans le retour JSON_

Gestion des erreurs remontées sur des équipements (pièces/modules)
@ThePiloo
Copy link
Contributor Author

ThePiloo commented Apr 4, 2024

J'ai amélioré la gestion des erreurs remontées en parallèle d'une réponse "status OK"

@zoic21
Copy link
Contributor

zoic21 commented Apr 5, 2024

Ok je comprends mieux je pensais qu'ils faisaient l'inverse.... Pas facile a suivre quand meme. Donc ca me va dis moi quand c'est bon.

Gestion de 2 nouveaux évènements webhook (topology_changed & home_event_changed/schedule)
Add wifi_strength info
@ThePiloo ThePiloo marked this pull request as ready for review April 6, 2024 10:22
@ThePiloo
Copy link
Contributor Author

ThePiloo commented Apr 6, 2024

je pense que c'est l'heure de tester

@zoic21 zoic21 merged commit ca2e1f4 into jeedom:beta Apr 6, 2024
5 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.

None yet

2 participants