Pour moi il y a forcément 2 points dans le temps : le point « historique » et l’heure actuelle
Vu qu’on parle d’un état, il faut forcément tirer la dernière barre entre le dernier point et maintenant non ?
J’ai fais la modification en ce sens.
J’ai ajouté (seulement pour les binaires) :
C’est push
Salut @pierre-gilles ,
Comme noté au-dessus, j’ai fais les changements tel que tu l’imaginais de ton côté.Add binary chart by Terdious · Pull Request #2094 · GladysAssistant/Gladys · GitHub
Hello @Terdious
Je t’ai fais une review fonctionnelle sur le sujet :
GladysAssistant:master ← Terdious:add-binary-chart
opened 11:44AM - 03 Jun 24 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [x] If your changes affects code, did your write the tests?
- [ ] Are tests passing? (`npm test` on both front)
- [x] Is the linter passing? (`npm run eslint` on both front)
- [x] Did you run prettier? (`npm run prettier` on both front)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [ ] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Taken from the original PR: #Binary graph #1948
Thanks to @callemand for all the work done
Pas encore de review technique tant que ce n’est pas bon fonctionnellement !
Hey @pierre-gilles ,
J’ai normalement appliqué et vérifié les remarques sauf :
J’ai tenté de reproduire mais, malgré des zooms / dezoom changement de résolution, etc je n’ai jamais réussi à avoir ton résultat. Par contre avant ma précédente modification j’avais le même souci, mais je l’avais corrigé avant ta review. Du coup j’ai quand meme réduit à 10 caractère seulement max au cas ou …^^
Et pour le power plug, je t’ai répondu sur github, mais je n’arrive pas à reproduire. J’ai ajouté un log sur les unités pour en voir plus.
Merci pour les correctifs !
Terdious:
J’ai tenté de reproduire mais, malgré des zooms / dezoom changement de résolution, etc je n’ai jamais réussi à avoir ton résultat. Par contre avant ma précédente modification j’avais le même souci, mais je l’avais corrigé avant ta review. Du coup j’ai quand meme réduit à 10 caractère seulement max au cas ou …^^
Ok, je n’arrive pas à reproduire non plus !
Je t’ai mis un commentaire pour le reste :
GladysAssistant:master ← Terdious:add-binary-chart
@Terdious Thanks for the fixes!
Here is the payload that causes the freeze: …
```
{"id":"e499dd5e-6482-4cbd-a73e-159d53c9b402","name":"Graph","selector":"graph","type":"main","visibility":"private","user_id":"275faa00-8a9c-4747-8fbe-417ddb966b16","created_at":"2024-06-10T08:20:20.784Z","updated_at":"2024-06-17T11:30:17.089Z","boxes":[[{"type":"chart","device_features":["mqtt-priseonoff"],"units":[null],"title":"Binaire","chart_type":"","interval":"last-day","display_axes":true}],[],[]]}
```
The response is 422 :
```
{
"status": 422,
"code": "UNPROCESSABLE_ENTITY",
"properties": [
{
"message": "\"[0][0].chart_type\" is not allowed to be empty",
"attribute": "boxes",
"value": [
[
{
"type": "chart",
"device_features": [
"mqtt-priseonoff"
],
"units": [
null
],
"title": "Binaire",
"chart_type": "",
"interval": "last-day",
"display_axes": true
}
],
[],
[]
],
"type": "Validation error"
}
]
}
```
It's just because I didn't select the "chart type" ^^ It's maybe not related to this PR, but I wonder if we could make a fix to avoid freezing in this situation...
I have another feedback, now the date in english is not correct:
Hola @pierre-gilles ,
¿Te parece bien para tu último comentario en esta PR:
Me parece que era el último punto a tratar.
¡Muy chulo!
Te he hecho 2 comentarios.
Creo que podríamos hacer un build de Cloudflare Pages después para que los usuarios de Gladys Plus puedan probarlo con su instancia, ya que es solo front debería funcionar (si los usuarios eliminan bien su gráfico « binario » después de la prueba).
Vale, bien para mí (también te he respondido ahí abajo)
Por otro lado, con los websockets, no tengo ningún problema, me quedé 2 minutos en la página y las actualizaciones se hacen cada 22s/30s sin refrescar la página… Por otro lado, en efecto, en la ventana emergente no se actualiza automáticamente, hay que mover el ratón.
¡Ah sí, sería genial de hecho!^^
¡Gracias por el cambio rápido! Te he respondido abajo, está bien para mí con los dos.
He hecho un PR para los builds de Cloudflare Pages (mantén tu PR):
master ← Terdious-add-binary-chart
ouvert 02:36PM - 30 Aug 24 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [ ] If your changes affects code, did your write the tests?
- [ ] Are tests passing? (`npm test` on both front/server)
- [ ] Is the linter passing? (`npm run eslint` on both front/server)
- [ ] Did you run prettier? (`npm run prettier` on both front/server)
- [ ] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [ ] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Please provide a description of the change here. It's always best with screenshots, so don't hesitate to add some!
La URL debería estar lista en unos minutos
Está disponible aquí:
https://terdious-add-binary-chart.gladys-plus.pages.dev
Recomiendo probarlo en un panel de control separado y borrar el panel de control después de la prueba (para no ensuciar tu producción)
guim31
30 Agosto, 2024 15:07
75
Así es como se ve en mi caso:
Me encantaría poder renombrar mi función (como en Dispositivo de la habitación) porque ahora, para colmo, es demasiado largo ^^
Por lo demás, todo está bien por mi parte !!
¡Gracias por la prueba, Guim31!
Al final no me sorprende.
@Terdious no sé si has jugado con las opciones del eje y
Típicamente puedes especificar un minWidth
Después bueno, esto ocupará espacio en el gráfico, no hay secreto, este tipo de gráfico se beneficiará mucho de la posibilidad de tener 2 o 1 columnas ^^
De lo contrario, hay que dejar al usuario la posibilidad de renombrar el dispositivo como en el widget « Dispositivos »
pierre-gilles:
Normalmente puedes especificar un minWidth
Después bueno, eso ocupará espacio en el gráfico, no hay secreto, este tipo de gráfico se beneficiará mucho de la posibilidad de tener 2 o 1 columnas ^^
Sí, y era exactamente eso, reducía tanto el gráfico que lo abandoné. Pero, de hecho, una mezcla entre esto y el renombrado puede manejar los casos abusivos.
¿Me baso en eso entonces? ‹ minWidth › + posibilidad de renombrar (la caja de edición va a ser enorme
¡Me encantaría ver eso sí! Efectivamente, va a ser una caja de edición enorme
He encontrado un primer compromiso que me parece interesante con ‹ minWidth › y algunas modificaciones. He cargado una de las cajas al máximo para probar realmente.
Acabo de hacer push, @pierre-gilles ¿Cloudflare se reinicia automáticamente o es una acción manual por tu parte?
Sigo con la posibilidad de renombrar, así que si es una acción de tu parte, quizá no sea necesario reiniciar una imagen antes. Te diré.
)
