Review des PR - J'ai besoin de vous!

2 projecteurs et 2 climatiseurs

Clairement :slight_smile:

N’hésite pas si tu as des retours, j’ai rajouté aujourd’hui 2 nouvelles améliorations à ce build:

  • Le merge du Zigbee2mqtt
  • Une amélioration assez importante des performances de Gladys Plus :slight_smile:

@Will9 j’ai repris le travaille sur Broadlink, il y avait une discussion sur la manière de gérer les features, et l’affichage. Je vais donc simplifier et généraliser pas mal de choses (et ajouter les appareils “clim”).

Sinon pour les autres,

Etat des lieux des PR en attente de revue / merge :

PR en attente de changement :

Quels sont les statuts de :

Je n’ai pris que les PR non draft, non WIP.

5 Likes

Wow génial @AlexTrovato :slight_smile: Merci pour la review complète !

C’est un problème de vulnerabilitié dans les dépendances globales, non liées au service :

found 30 vulnerabilities (28 low, 2 high) in 1184 scanned packages
 run `npm audit fix` to fix 29 of them.
 1 vulnerability requires manual review. See the full report for details.
Error: Process completed with exit code 1.

A voir pour faire un fix sur Master…

Je veux bien que quelqu’un regarde pour m’aider à gérer correctement l’erreur mal catcher par la librairie :wink:

Tu es sûr? On a pas ça sur master en tout cas, rebase master je pense ça devrait fixer ton erreur :slight_smile:

En effet, le rebase corrige ça :wink:

2 Likes

@VonOx en vrai tu prépares ta reconvertion professionnelle ? J’ai fait une passe sur ta PR, c’est pas mal !!! Mais j’ai fait quelques retours. Je pense qu’il sera sympa que tu fournisses la commande docker (docker-compose) pour lancer pi-hole avec des ports non conflictuels avec Gladys, c’est pas compliqué, mais pas trivial pour autant.

1 Like

Oula non ! :sweat:. J’aurais du dire merde au collège quand on m’as dit qu’il n’y avait pas de débouché dans le logiciel ( véridique ) et faut vraiment que je mette à jour ma présentation car j’ai un parcours pro assez particulier. ( dessinateur/concepteur mécanique maintenant consultant/integrateur PLM ).
Je te cache pas que celà m’aurait plus mais par rapport à vous ( les devs) il y’a un gap qui me semble insurmontable !


Bref revenons en à cette PR. Comme tu as pu le remarquer , il me faut une base, des exemples pour développer, j’ai pioché dans des PR en cours, dans l’existant pour sortir un truc qui marche à peu près.

Il y’a des choses que je ne maitrise pas encore et ça se voit mais bon , on apprend tous les jours !

Merci d’avoir tester et merci pour les feedbacks !

Si tu veux des réponses techniques à mes remarques, on peux échanger en MP. Je te dirais quoi/comment faire, pourquoi le faire comme ça et tout et tout… :wink:
Je ne dis pas non à un nouveau dev :smiley:

Tu es entrain de dev le service linky ?

Non qu’est ce qui te fait dire ça ?

ça date un peu mais bon. je me suis fait une fausse joie lol

Pour consommer l’api enedis il faut être une entité juridique, ça sera probablement proposé via Gladys plus.