Característica Termostato completa

No, eso no existe. Podrías hacer una solicitud de función para esto.

Yo haría un termómetro virtual promedio y en una escena recuperas los valores de cada termómetro y luego aplicas un cálculo para obtener el promedio

¡Listo, probado!

En resumen, ¡es realmente, realmente, realmente genial!

Dos cosas que me han dado problemas. Probado solo en móvil Android:

  1. En el modo planificación, si hago clic en la hora, el teclado no aparece, ni la selección de hora (y el icono del reloj es demasiado poco visible en modo oscuro)

  1. Si pongo en ‹ Apagado ›, por mucho que vuelva al planificador, el botón no se actualiza (debería volver al modo eco o noche o… (En realidad, a esta hora no había definido nada, así que me pregunto al escribir estas líneas qué está definido por defecto. ¿Eco? Ah, no. Al revisarlo, es ‹ apagado › por defecto, así que supongo que todo es normal…)

Estoy muy entusiasmado y ansioso por ver este módulo en producción, ¡ahora mismo! ¡Enhorabuena, es realmente genial (¿ya lo he dicho?)

Muchas gracias por tu valiosa opinión… Claude ha sudado a veces…

De acuerdo con el punto 1, haré (Claude) una corrección.

En cuanto al punto 2, sí, si no has definido nada en tu planificación es normal. Después, en cuanto hayas pulsado el botón, como el mensaje sigue, puede que te haya perturbado. Estoy pensando qué podría hacer.

Gracias de nuevo por tu opinión

Tienes razón, ese es el problema. El mensaje debería volver a ‹ detenerse ›, como en los otros modos :wink:

@GBoulvin, ¿cuál es tu navegador en tu móvil Android?
Claude me dice que probablemente se deba al navegador.

¿Te serviría si pongo el selector como en las escenas?

Es Chrome.

El selector actual está muy bien, solo me gustaría poder escribir la hora manualmente, con el teclado, o tal vez simplemente que el selector se abra al hacer clic en la hora y no solo en el icono.

Puedo volver a probarlo al final del día con otro navegador si quieres…

Ah, vale, no lo había entendido en absoluto.

Me gustaría que volvieras a probar con otro navegador si es posible para ti.
Gracias

@Will_71 ¡Mantenme informado en cuanto creas que está listo para salir! :grinning_face_with_smiling_eyes: La idea sería apuntar a un lanzamiento a más tardar la próxima semana, para que yo aún esté aquí durante las dos primeras semanas de vida de la funcionalidad. Por cierto, me quedan entre 9 y 19 días de presencia en Gladys, ¡dependiendo de la fecha del parto!

Si te parece bien, propongo hacer un merge hoy en master y hacer un build :dev. Si hay comentarios durante el fin de semana, haré un merge de los comentarios el lunes, si no, se va a producción el lunes :slight_smile:

Estoy un poco bajo presión con mi trabajo hoy, pero puedo avanzar en ello este fin de semana. Quedan los 2 puntos que @GBoulvin ha señalado.

@Will_71 He lanzado mi Claude para analizar los comentarios de @GBoulvin y sus comentarios son legítimos. El diagnóstico completo está disponible aquí: feat(thermostat): add a thermostat integration with dashboard widget by William-De71 · Pull Request #2988 · GladysAssistant/Gladys · GitHub

Si no estás disponible hoy, te propongo hacer yo mismo las correcciones, luego fusionar en master, y luego si tienes más correcciones que hacer este fin de semana, puedes hacer más PRs y se enviarán el lunes, ¿qué opinas? :slight_smile:

Sí, adelante, por hoy permitirá avanzar ya que no podré ocuparme.
Podría mirar con más detalle este fin de semana, pero para mí solo quedaban estos 2 puntos.

Una vez que hagas el merge en master, también podré preparar la continuación con la gestión de los hilos de los pilotos… mis PRs están listos para ser rebaseados y luego probados en la práctica.

¡Hola a todos!

Este tema está ahora en desarrollo.

Se ha abierto una PR para corregir el botón « Volver al planificador » en un intervalo de Parada, y el selector de hora del planificador en Android:

No duden en seguir la PR, probar (opcional, especialmente para pequeñas solicitudes) y dar sus comentarios aquí si es necesario.

Al menos voy por mi tercera pasada :joy:

Hice que Claude hiciera una pasada de UX en tiempo real + varios code reviews con Fable 5.1.

Tardó 45 minutos y tomó muchas capturas de pantalla, y encontró algunos errores.

¡Los está corrigiendo!

No me jodas todo mi trabajo :sob:.

Me río, si podemos corregir otros errores que no hemos visto y que el termostato sea perfecto​:+1:

Espero no haber roto nada :joy:

Avísame cuando hayas avanzado/terminado para que pueda probarlo por mi parte

Lo que voy a hacer es que de todos modos voy a hacer un merge en master tan pronto como esté satisfecho hoy, así que puedo hacer un build que agrupe todos los grandes cambios de esta semana, y luego si tienes comentarios, podrás hacerme pequeñas PRs correctivas este fin de semana que integraré el lunes :slight_smile:

¡Ya está fusionado en master!