Psoy
20. November 2023 um 21:23
1
Guten Abend @pierre-gilles Nur ausführen, wenn der Schwellenwert überschritten wurde (und nicht bei jedem gesendeten Wert)
Will_71
20. November 2023 um 21:49
2
Selber Bug, aber wenn schon angehakt, kann man es nicht mehr abhaken
Ein Rückfall (dieser Bug war doch schon behoben, glaube ich)
Will_71
20. November 2023 um 23:09
5
@pierre-gilles ,
guim31
21. November 2023 um 07:48
6
Ich stimme zu. Ich habe im Update-Ankündigungsthread gepostet, bevor ich eure Nachrichten gesehen habe…!
b3n.0
21. November 2023 um 08:31
7
Ja, bei mir ist es genauso und auch bei Gladys+…
Kann jemand ein GitHub-Issue erstellen?
Edit: Ich habe das Thema verschoben und das andere geschlossen
guim31
21. November 2023 um 17:49
9
Ja, du hast recht, wir haben uns ein bisschen verzettelt!
Will_71
21. November 2023 um 18:36
10
Ich werde mich darum kümmern, das Issue zu erstellen
Will_71
21. November 2023 um 20:50
11
Hier also das Issue und der PR
ouvert 08:36PM - 21 Nov 23 UTC
fermé 10:00AM - 27 Nov 23 UTC
In scene trigger on device change, threshold is not clickable. The same bug as t… he one already fixed in June
https://community.gladysassistant.com/t/impossible-de-cocher-la-case-executer-seulement-lorsque-le-seuil-est-passe/8515
master ← William-De71:scene-trigger-threshold
ouvert 08:44PM - 21 Nov 23 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
PR to fix threshold not clickable on scene trigger
https://github.com/GladysAssistant/Gladys/issues/1940
Will_71:
Du coup l’issue et la PR
Danke für die PR! Tatsächlich löst das das Problem, aber meiner Meinung nach ist das Problem tiefer als das. Ich würde mich nicht wundern, wenn das Problem in Zukunft wieder auftaucht.
Ich glaube, das Problem kommt von einem Preact-Bug, ich habe diese GitHub-Issue gefunden, als ich ein bisschen gesucht habe:
ouvert 11:39AM - 24 Aug 20 UTC
fermé 10:43AM - 16 Jul 26 UTC
duplicate
compat
known issue
### Reproduction
With this example, React creates a controlled input while Pr… eact creates a uncontrolled input.
```js
import React from "preact/compat";
function App() {
return <input type="checkbox" checked={true} />;
}
```
In fact, I can't figure out how to make a controlled checkbox at all.
Not sure if this is a duplicate of #2625 or #1899
### Steps to reproduce
https://codesandbox.io/s/preact-input-uncontrolled-qis0i
Click on the checkbox
### Expected Behavior
Clicking on the checkbox shouldn't change its state (= what React does).
### Actual Behavior
The checkbox can be toggled. `checked={true}` is essentially ignored.
Ich habe die PR trotzdem gemerged, da es so aussieht, als würde es die Situation entblocken, aber es muss beobachtet werden.
Will_71
22. November 2023 um 19:36
14
Es ist offensichtlich, dass es nicht normal ist, das so ändern zu müssen. Das sollte man im Auge behalten.
Ich habe gerade den Fix auf Gladys Plus bereitgestellt (da es sich um einen Frontend-Fix handelt, ist er ab sofort auf Gladys Plus verfügbar)
Eine Version für Gladys Local kommt später
Psoy
23. November 2023 um 17:46
16
guten Abend @pierre-gilles
Behobenes Problem in Gladys Assistant 4.31.1:
Salut à tous !
Je viens de sortir Gladys Assistant 4.31.1, une version mineure avec quelques correctifs:
Correction d’un bug assez critique qui empêchait de cocher la case « Exécuter seulement lorsque le seuil est passé » dans les scènes ! Bug corrigé par @Will_71 , merci du fix @Will_71 pour le fix !
Amélioration des traductions. Merci @cico…
Man sollte wirklich neue Themen erstellen, diese langen Themen sind wirklich kompliziert 