Es wäre gut, eine Szene deaktivieren zu können, ohne sie zu löschen.
Heute muss ich meine Auslöser löschen und sie dann neu erstellen, wenn ich zurückkomme (es sei denn, ich habe eine Funktion übersehen).
Nicht mehr abstimmen, sondern +1 geben
Lokkye
13. Februar 2021 um 20:15
5
Hallo, ich habe einen kleinen PR für dieses Thema erstellt
master ← callemand:disable_scene
ouvert 08:13PM - 13 Feb 21 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)
- [ ] 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).
- [X] Did you add fake requests data for the demo mode (`front/src/config/demo.json`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
### Description of change
Adding disable scene and change start button by test
Könnt ihr mir euer Feedback geben?
Hallo @Lokkye !
Danke für dein Pull Request
Ich denke, es gab ein Missverständnis der Spezifikation, dein PR erfüllt meiner Meinung nach nicht genau den Bedarf hier.
Ich habe einen Kommentar auf dem PR hinterlassen:
master ← callemand:disable_scene
Hey! Thanks for your PR.
I'm not really sure this is the behavior we want.
…
It's weird to me that a disabled scene could not be tried...
To me, the "disable scene" feature is more a "disable trigger for this scene" feature, not a "disable the scene completely".
I think people want to disable a scene while they are still building it, or if they want to pause the automatic start of the scene (while they are in holiday for example). But still, I think that people still want to be able to do manual runs.
What do you think about that?
Ich fordere ebenfalls diese Funktion zum Deaktivieren/Wiederaktivieren einer Szene, da ich meine Szenen ebenfalls jedes Mal löschen und neu erstellen muss, wenn ich einen freien Tag oder ähnliches habe.
Hallo zusammen!
Da @Lokkye nicht geantwortet hat, habe ich die Entwicklung übernommen
Ich habe das Design, das gemacht wurde (mit einigen Änderungen), verwendet, aber die Server-Implementierung und die Tests neu geschrieben.
Ich habe gerade die Funktion in master gemerged, sie wird in der nächsten Veröffentlichung enthalten sein!
Danke @Lokkye für die anfängliche Entwicklung
https://github.com/GladysAssistant/Gladys/pull/1229
Zur Info, ich habe gerade einen Dev-Build gestartet, der diese Funktion beinhaltet!
Der Build: https://github.com/GladysAssistant/Gladys/actions/runs/1029251642
Sobald der Dev-Build fertig ist, bin ich für jedes Feedback offen!
Edit: es ist live!
Die Funktion ist jetzt in Gladys Assistant v4.5 verfügbar!
Salut à tous !
On commence la rentrée à fond, avec la release de Gladys Assistant v4.5, une mise à jour assez majeure qui apporte de belles fonctionnalités comme les tableaux de bords multiples !
Pour voir toutes les nouveautés de cette version, j’ai posté un article assez complet sur le blog:
Testez Gladys Plus gratuitement
C’est quelque chose qui manquait: un programme d’essai de Gladys Plus! Il est désormais possible de tester Gladys Plus pendant 14 jours, entièrement gratuitement (pas …
