Variable in ID-Namen verwenden?
-
Okay, falls ich es zum laufen bringe soll ich dann ein Pull Request auf Github für ioBroker.javascript machen?
@apollon77Oder lieber den Code hier posten damit es jeder selbst hinzufügen kann?
@paul53 Ist die funktion getIdByName() global oder von dir selbst erstellt. Ansonsten könnte man dafür auch ein eigenes Block-Element erstellen.
-
@thewhobox sagte in Variable in ID-Namen verwenden?:
Okay, falls ich es zum laufen bringe
Ein bisschen mehr Optimismus
soll ich dann ein Pull Request auf Github für ioBroker.javascript machen?
Wäre, glaub ich, der richtige Weg
Kann man auch noch andere Baustein-Wünsche an Dich richten?
(Vooorsicht mit der Antwort ) -
@thewhobox sagte:
Ist die funktion getIdByName() global oder von dir selbst erstellt.
Die Blockly-Funktion ist eine von mir erstellte Funktion, die die Funktion getIdByName(name) des Javascript-Adapters verwendet.
-
@rantanplan sagte in Variable in ID-Namen verwenden?:
Kann man auch noch andere Baustein-Wünsche an Dich richten?
jetzt erstmal schauen ob und wie gut der jetzt geht. Danach bin ich offen für alles.
Zum laufen gebracht habe ich ihn schonmal, nur ersetzt er jetzt den alten. Möchte aber beide haben, da per Auswahl oft doch bequemer ist. -
@thewhobox sagte:
da per Auswahl oft doch bequemer ist.
Den Select ID Block kann man doch einfach einhängen.
-
@paul53 Das ist auch eine sehr gute Idee.
Aaaber damit werden doch auch alle bereits erstellten Skripte geändert. Man müsste also dann in jedem Skript das "Select ID"-Element anhängen und erneut zuordnen. -
@thewhobox sagte in Variable in ID-Namen verwenden?:
@rantanplan sagte in Variable in ID-Namen verwenden?:
Kann man auch noch andere Baustein-Wünsche an Dich richten?
jetzt erstmal schauen ob und wie gut der jetzt geht. Danach bin ich offen für alles.Gute Antwort
Zum laufen gebracht habe ich ihn schonmal, nur ersetzt er jetzt den alten. Möchte aber beide haben, da per Auswahl oft doch bequemer ist.
Der Alte sollte auch drin bleiben, abwärts kompatiebel.
-
@thewhobox sagte:
damit werden doch auch alle bereits erstellten Skripte geändert.
Bei bereits besehenden Blocklys benötigt man natürlich beide Varianten.
-
So es laufen nun endlich beide parallel:
Jetzt muss ich nur noch herausfinden wie man ein Pull Request macht xD -
@thewhobox Ja ist denn schon Weihnachten?
Schreib mal was in dieses issue
Da wird sich hoffendlich jemand melden. -
@thewhobox sagte in Variable in ID-Namen verwenden?:
Jetzt muss ich nur noch herausfinden wie man ein Pull Request macht xD
in github das Original forken
ImFork ändern und dann "New Pullrequest" auslösen -
@rantanplan Erledigt.
Ebenso meinen ersten Pull Request veröffentlicht@Homoran Ah okay, den Button hab ich nicht gesehen. Ich hab das jetzt ein bisschen komplizierter hinbekommen^^
-
@thewhobox sagte in Variable in ID-Namen verwenden?:
Ebenso meinen ersten Pull Request veröffentlicht
Cool. Gratulation!
-
Kleines Update:
Pull Request wurde gemerged. Sollte also im nächsten Update enthalten sein.
Oder ihr ersetzt die Datei "admin/google-blockly/own/blocks-system.js" und macht danach ein Upload. -
Erneutes Update: Hab gerade gesehen, dass die Änderungen für den Block vor 3 Tagen per Commit wieder gelöscht wurde.
Weiß nicht wieso, da es vor 6 Tagen ja noch gemerged wurde... NajaWer es trotzdem braucht kann den Block auch selbst hinzufügen, wird allerdings bei jedem Update dann überschrieben und muss erneut hinzugefügt werden. Die zu ändernden Dateien findet ihr hier.
-
Ich fände es besser wenn der "offizielle" Weg eingehalten würde.
Sonst tauchen hier im Forum irgendwelche Fehlerfragen auf, die niemand oder nur wenige nachvollziehen können.Ich frage in Git mal nach. Solltest Du vielleicht auch tun.
-
@rantanplan Hab gerade beim Commit nachgefragt.
-
JS 4.1.12:
-
Und es funktioniert sogar!
Vielen Dank @thewhobox -
@rantanplan Gerne doch
Hab auch mal noch eins für getIdByName() gemacht. Könnte für einige vll auch interessant sein.
Oder was bräuchtest du noch so?
