opened
b1rdhous3
username_0
**1.** Tell us something about your changes.

Moved Token Management to Network Settings, as wished by @username_1 in PR https://github.com/hyperion-project/hyperion.ng/pull/578

**2.** If this changes affect the .conf file. Please provide the changed section

**3.** Reference an issue (optional)

Note: For further discussions use our forum: forum.hyperion-project.org
Moved Token Management to Network Settings
issue
created
Paulchen-Panther
username_1
Ein paar Sachen sind da noch. :grinning: 
Bei mir lassen sich keine Token erstellen!
comment
created
b1rdhous3
username_0
Dein "123123123" erfüllt die Mindestlänge nicht. Du musst noch ein Zeichen mehr machen. 

Hm, war das nicht auf der anderen Seite auch so mit dem Button? Aber ja, geb ich dir recht, das sollte man noch mal optimieren dort.
comment
created
Paulchen-Panther
username_1
https://github.com/username_0/hyperion.ng/blob/WebUiToken/assets/webconfig/js/content_effectsconfigurator.js#L96

Sowas in der art
comment
created
Paulchen-Panther
username_1
Und wenn ich die Erklärungen im General Bereich ausschalte ist die Box etwas verschoben.
https://prnt.sc/onwhg0
comment
created
Paulchen-Panther
username_1
Könntest du wenn du schon mal dabei bist, die Erklärung für die Token mit erstellen? Das wäre echt supi.
comment
created
b1rdhous3
username_0
Was genau möchtest du mir hiermit sagen? Dass der Token Liste dynamisch aus und eingeblendet werden soll in Abhängigkeit zur "API" Checkbox? Das sollte bereits so sein. 

Komme heute nicht mehr dazu weiterzumachen.
comment
created
Paulchen-Panther
username_1
Man könnte doch bestimmt auf dieses Event (change keyup) reagieren um eine Meldung anzeigen zu lassen wenn der String noch zu kurz ist.
comment
created
b1rdhous3
username_0
Achso, ja sowas geht. Kann ich einbauen. 

Hints kann ich auch erstellen.

Das das ohne Hints nicht ganz bündig ist, liegt daran, dass ohne Hints keine Rows erzeugt werden und ich für das Element immer eine Row erstelle. Entweder erstellen wir trotz den ausgeschalteten Hints Rows, oder ich muss schauen, dass ich das Verhalten dort bei der Token Liste dynamisch anpassen kann, sodass das dann auch ohne eine Row angelegt wird.
comment
created
Paulchen-Panther
username_1
Meine HTML Kenntnisse reiche gerade mal dazu es lesen und verfolgen zu können.
Ich vertraut dir da voll und ganz.
comment
created
b1rdhous3
username_0
@username_1 Hab deine Wünsche mal umgesetzt. 
Auch habe ich eine Explanation hinzugefügt. 

Ne ganze Hilfstabelle hab ich nicht gemacht. Wüsste auch nicht wozu.
comment
created
Paulchen-Panther
username_1
Könntest du deinen PR bitte noch mal neu erstellen. Irgendwas ist bei meinem letzten rebase schief gelaufen. Sorry dafür
comment
created
b1rdhous3
username_0
Bekomme die Meldung dass der PR dann nicht automatisch zusammengeführt werden kann. 
Bin bisschen mit meinen Git Know How am Ende wie ich nun am besten fortfahre.
comment
created
Paulchen-Panther
username_1
Das hilft nur ein rebase deiner seite her. Also erst den localen master branch auf aktuellen stand bringen und danach den WebUiToken branch mit dem aktuelliesierten master rebasen.
comment
created
b1rdhous3
username_0
Mein lokaler Master müsste in dem Fall auf dem aktuellen Stand deines api_auth sein, korrekt?
comment
created
Paulchen-Panther
username_1
Muss nicht unbedingt der master sein. du kannst auch einen neuen erstellen auf basis meines api_auth branch. und dann deinen WebUiToken branch mit dem neuen rebasen.
comment
created
Paulchen-Panther
username_1
Das ging ja fix
comment
created
b1rdhous3
username_0
So hab ich das nun auch gemacht. 
Kannst du den schließen, damit ich den neuen öffnen kann?
comment
created
Paulchen-Panther
username_1
Den PR? Warum schließen? Sollte doch jetzt wieder stimmen!
comment