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

Calcul et publication de statistiques sur le coût avec un tarif unique #24

Closed
wants to merge 2 commits into from
Closed

Calcul et publication de statistiques sur le coût avec un tarif unique #24

wants to merge 2 commits into from

Conversation

olivier91ou33
Copy link

Ajout d'un paramètre de config permettant d'indiquer le prix du kWh de consommation et production
Ajout d'une méthode de publication des statistiques de coûts/revenus correspondants aux stats de consommation/production

@bokub
Copy link
Owner

bokub commented Feb 13, 2024

Merci beaucoup ! J'essaie d'y jeter un œil très vite

@bokub
Copy link
Owner

bokub commented Feb 21, 2024

Hello

Merci pour cette pull request mais ton code n'est même pas écrit avec du Typescript valide:

  • J'ai du rouge dans mon IDE
  • La CI/CD m'indique que la commande de build ne fonctionne pas (plus le lint qui ne passe pas mais c'est un détail)

image

Du coup, si j'ai bien compris, tu me demandes de review du code que tu n'as même pas testé ? C'est un peu audacieux de croire que ça va marcher du premier coup si tu veux mon avis

@olivier91ou33
Copy link
Author

effectivement, j'ai fait la PR sur la mauvaise branche. Il manque un commit pour corriger float en number
J'ai testé en exécutant les fonctions au debugger, Pas l'environnement pour faire un test global intégré à HA

@olivier91ou33 olivier91ou33 deleted the base_tarif_support branch February 28, 2024 14:57
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.

2 participants