Funktionsbeschreibung
Ich kann eine Szene nur zwischen 18:00 und 5:00 Uhr auslösen, der Test erlaubt es nicht.
Und leider wird das Hinzufügen einer zweiten Bedingung als „UND“ interpretiert, was immer falsch ist.
Ich schlage die Hinzufügung eines „ODER“-Buttons vor, wie in den anderen Bedingungsblöcken, speziell in den Zeitbedingungen.
guim31
30. Januar 2022 um 09:53
2
Ich unterstütze das voll und ganz für die zeitlichen Bedingungen, die sich über zwei Tage erstrecken (also meistens nachts).
Da so viele Leute diesen Fix angefragt haben, habe ich mir Zeit genommen, um das dringend zu beheben
Ich schlage diesen PR vor, ich bin offen für Feedback:
master ← add-or-condition-check-time-scene
ouvert 09:36AM - 31 Jan 22 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?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [x] 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](https://community.gladysassistant.com/) for testing before merging.
- [x] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [x] 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).
- [x] 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
This let the user use conditional time check when after time is before the before time.
VonOx
31. Januar 2022 um 16:51
4
Wie bei @cicoub13 funktioniert es bei mir auch.
guim31
31. Januar 2022 um 17:03
5
Wenn man Druck macht, passiert so etwas …
Das wird am Ende in Demonstrationen enden!!
Ich kann nicht sagen, ob es funktioniert, aber ich weiß nicht, wie man es testet (muss man eine Art beta-Instanz von Gladys haben?)
Vielen Dank auf jeden Fall für diese Schnelligkeit!!
VonOx
31. Januar 2022 um 17:45
6
Nein, das ist wirklich manuell ab dem Quellcode.
guim31
31. Januar 2022 um 18:00
7
Ah ja okay, dann bin ich wohl dran ^^ danke
Der Build ist durchgelaufen, die Version ist verfügbar
Ich freue mich über Feedback zu diesem Fix, wenn alles gut ist, können wir das Thema schließen.