search

ldotlopez / ha-ideenergy

ideenergy integration for home-assistant
GNU General Public License v3.0
74 stars 13 forks source link

Update Instant data missing from previous Commit #5

Closed ad-ha closed 2 years ago

ad-ha commented 2 years ago

En el pull anterior se me olvidó añadir las líneas de actualización del atributo. El patch-1 y patch-2 van juntos.

ldotlopez commented 2 years ago

Hola Álvaro,

¿Podemos unir el PR #4 y PR #5 sobre un único parche y a partir de ahí lo miramos?

A mi me ha tocado hacer algún experimento porque no lo tenía muy claro, te dejo lo que he hecho:

git checkout patch-1 # Cambiar al patch-1, PR4
git rebase patch-2   # Aplicar los cambios de patch-2 sobre la rama actual, patch-1
git push             # Subirá los cambios (los dos commits) a tu rama patch-1, el PR #5

Así cerramos este PR y continuamos en el otro

ad-ha commented 2 years ago

Buenas

Intentaré hacerlo. Que sólo uso Git en la web. Si no cierro los dos pull y creo uno nuevo con todo junto.

ad-ha commented 2 years ago

Creo que está. El PR #4 creo que se puede cerrar.

ldotlopez commented 2 years ago

Gracias Álvaro.

Me gustaría comprobar si puedo extraer más datos del API de i-de además de las lecturas acumulada y real para empaquetarlo todo en un solo atributo… pero como veo que no estoy pudiendo sacar tiempo apruebo el PR y ya lo miraré.

ad-ha commented 2 years ago

Hola Luis, Gracias a ti. Lo vamos mirando. Mientras, si te parece, abriré nuevos "issues" con las propuestas de desarrollos que me han ido pidiendo en mi integración y lo comentamos. Saludos