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

Configuración del sensor a través de HA UI #44

Merged
merged 4 commits into from
Jan 4, 2022
Merged

Conversation

tijunoi
Copy link
Contributor

@tijunoi tijunoi commented Dec 28, 2021

Hola, dejo aquí esta implementación de la configuración por UI de HA. Sé que estaba en el roadmap y quería ayudar, y como no podía ayudar con lo demás, pues me he adelantado al roadmap 😬

La he probado en mi homeassistant y funciona. No tenía la integracíon configurada previamente, así que no he probado con una instancia que ya tuviera la integración configurada.

La implementación no migra automáticamente, pero creo que eliminando del YAML config y añadiendo la integración por la UI, debería funcionar, ya que el sensor generado va a ser el mismo.

Pd: la integración con el panel de energía va muy bien!

This implementation does not allow for editing the config entry.
It also does not implement migration of the YAML config entry to
the new UI config entry, but since data is stored by `scups`, removing
it from configuration.yaml and setting it up from the UI should work.

`debug` config is still taken from yaml config, since it is global and not per
CUPS.
@danieldabate
Copy link

danieldabate commented Dec 28, 2021

Hola! Estoy intentando probar funcionalmente la nueva integración, desde cero, pero sin éxito. Los pasos que seguí son:

  1. Agregar repositorio en HACS y agregar el componente del branch dev
  2. Agregar la integración con edata desde la UI, ingresando username, password y CUPS.

Resultado: Se agrega la integración con el nombre autogenerado a partir de mi CUPS, pero no encuentro ningun dispositivo o entidad que tenga la información del consumo.

Me está faltando algún paso?

Gracias!!

EDIT: Me faltaba agregar el sensor edata al configuration.yaml y hacer reload. Ahora si! A probar :D

@tijunoi
Copy link
Contributor Author

tijunoi commented Dec 28, 2021

EDIT: Me faltaba agregar el sensor edata al configuration.yaml y hacer reload. Ahora si! A probar :D

@danieldabate Sí, de hecho es algo que me he dejado.
El código en async_setup_platform se ejecuta solo si hay algun sensor con platform edata en configuration.yaml, y he dejado ahí el init de websockets, ya que era global y no por entry. Tengo que buscar el equivalente de ese método para UI config entries y ya no haría falta añadir nada en configuration.yaml

@uvejota
Copy link
Owner

uvejota commented Dec 29, 2021

Pero qué grande, voy a solucionar primero un par de bugs de la integración con el panel de energía y pruebo esto :-)

Copy link
Owner

@uvejota uvejota left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lo veo cojonudo. Sólo faltaría implementación de la migración de configuración desde el yaml si la hubiera. Voy a investigarlo estos días a ver si saco algo de tiempo.

¡Mil gracias!

@tijunoi
Copy link
Contributor Author

tijunoi commented Dec 29, 2021

@uvejota Yo también estoy investigando. En la docu oficial solo he encontrado migración entre versiones de config entries, no de yaml a config entry. Si no recuerdo mal la integración philips hue (entre otras) tenía migración automática durante unas versiones en concreto. Si lo encuentro actualizaré la PR.

@tijunoi
Copy link
Contributor Author

tijunoi commented Jan 4, 2022

@uvejota Listo! Lo he probado con config válida y con config inválida. Se importará automáticamente. Si un CUPS ya está configurado no lo duplicará.

Edit: la documentación no la puedo actualizar ya que está en el wiki y no en el repo.

@uvejota uvejota merged commit 2382c12 into uvejota:dev Jan 4, 2022
@uvejota
Copy link
Owner

uvejota commented Jan 4, 2022

Es una maravilla, muchas gracias.

Lo dejo un tiempo en dev y saco release a lo largo del mes (quiero ver si me da tiempo a meter un par de cosillas más) :-)

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

3 participants