Hallo zusammen,
Ich arbeite an einer oft gewünschten Funktion, die meiner Meinung nach vielen nützen wird: die Möglichkeit, Variablen in Szenen zu injizieren!
Ein kleiner Vorgeschmack auf die Funktion:
Edit: der PR:
master ← inject-scene-variables-in-other-boxes
ouvert 02:24PM - 18 Jan 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)
- [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.json`) 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!
(Es ist WIP)
Ich habe gerade einen ersten POC der Funktion auf GitHub gepusht. Ich bin offen für alle Rückmeldungen zur PR/Tests/UX-Feedback
master ← inject-scene-variables-in-other-boxes
Hi @cicoub13 @VonOx @atrovato, I would love a review from you on this PR!
Th… is is how it looks like:
https://user-images.githubusercontent.com/7365207/104939530-d229c100-59b0-11eb-93e1-87bf4c4bcfe1.mp4
Is the shortcut straightforward? (having to type " {{ " to open the variable list ?) Should we use another trigger ? Like "#" ?
This feature will be extended to basically all input field in scenes, for exemple in the HTTP request it could be super useful to automatically inject a variable into a POST body :)
Hallo. Ich habe mir die PR angesehen und einen Fehler bei der Initialisierung von tagify gefunden
Uncaught (in promise) TypeError: _this.props.action.text is undefined
initTagify SendMessageParams.jsx:102
refreshVariables SendMessageParams.jsx:78
promise callback*SendMessageParams/_this.refreshVariables/< SendMessageParams.jsx:76
refreshVariables SendMessageParams.jsx:49
componentWillReceiveProps SendMessageParams.jsx:138
Ich habe es auf alle möglichen Arten versucht, mit verschiedenen Geräten oder Aktionen in den vorherigen Gruppen
Ach Mist! Danke für das Feedback. Ich halte dich auf dem Laufenden, wenn es behoben ist
@cicoub13 normalerweise sollte es jetzt gut sein, ich habe einen Fix gepusht!
Lass mich wissen, wenn du weitere Fehler siehst. Zögere nicht, die Funktion zu testen, sie muss robust sein!
VonOx
22. Januar 2021 um 10:26
7
Hallo, ich habe den Branch für den Test ausgecheckt.
Auf Chromium/Firefox passiert nichts, wenn ich {{ eingebe
Da dachte ich mir « Vincent, du hast Mist gebaut, du bist nicht auf dem richtigen Branch »
Anscheinend doch ^^
Funktioniert das nur für die Sensoren?
Vielleicht ist das nicht ganz klar, aber es muss unbedingt eine Box geben, die eine Variable setzt, um Variablen verwenden zu können!
Guter Hinweis
Hier ein Beispiel einer Szene:
Ideen, wie man das klarer machen könnte?
VonOx
22. Januar 2021 um 10:32
9
Ah ja klar, das funktioniert besser ^^
Kleiner Bug in FF
Ich denke, man sollte (wenn nichts angezeigt wird ^^) angeben, dass man zuerst den Wert des Geräts abrufen muss.
VonOx:
Kleiner Bug in FF
Mist. Ich schau mal, ist ja eine fertige Lib, die ich nutze, hoffentlich ist es nicht schlimm ^^
VonOx:
Ich denke, man sollte angeben (wenn nichts angezeigt wird ^^), dass man zuerst den Wert des Geräts abrufen muss.
Gute Idee!
Bei mir funktioniert alles auf Firefox, ich glaube, das ist ein Problem, weil deine Box sehr lang ist!
Bearbeitung: Es ist tatsächlich ein Problem mit der Textgröße.
Man könnte die Box sogar größer machen, um größeren Variablen Platz zu bieten:
Vorher:
Nachher:
Ich denke, dieser Feature-Request kann geschlossen werden
