uvejota / edistribucion

Home Assistant custom integration for e-distribución
GNU General Public License v3.0
20 stars 6 forks source link

EDSConnector.get_meter() API endpoint seems to be no longer working #27

Closed poveden closed 3 years ago

poveden commented 3 years ago

¡Hola! Antes que nada, quiero agradecerte por este componente. Me parece súper útil.

Ayer instalé este componente en reemplazo del de @jagalindo, pero desde entonces las métricas del ICP han dejado de funcionar.

Escarbando un poco en el código, y comparando con el comportamiento desde mi navegador, he notado que el endpoint para obtener las métricas es diferente:

He hecho el cambio "a fuego" en mi instalación, y ahora tengo métricas. 🎉

¿Quieres que te haga un PR con el cambio? No soy ni remótamente experto en Python, pero creo que añadir unos caracteres a un string está dentro de mis capacidades. 😛

jagalindo commented 3 years ago

Buenas @poveden.

Me alegro que hayas podido echarle un vistazo, yo espero dedicarle algunas horas durante agosto para integrar varios cambios. Si puedes te agradezco el pr y con eso lo metemos en el master.

Un saludo!!

uvejota commented 3 years ago

Hola a ambos (por cierto @jagalindo gracias por tu implementación original jeje).

Gracias por la propuesta @poveden, ahora mismo me pillas de vacaciones y hasta mañana no voy a poderle echar un ojo. Pero pinta genial, ¡ojalá funcione!

En cualquier caso, estoy reescribiendo el conector por completo y no tengo claro si lanzar un hotfix para esto mientras lo termino, o directamente hacerlo en la versión final. Así que lo del PR como prefieras, como tengo tantos otros cambios pendientes de lanzar puedo hacerlo yo mismo :-)

¡Gracias de nuevo por la sugerencia!

uvejota commented 3 years ago

Cambiado en d4d36782c31a457063c19719045a7b987095f839

Y disponible en v1.1.2.1

¡Gracias por la sugerencia!