search

rowe42 / lhm_animad_admin_html5

0 stars 6 forks source link

Überflüssiges Event detail aus Polymer-Doku #283

Closed peter-mueller closed 6 years ago

peter-mueller commented 6 years ago

Hallo,

bei: https://github.com/xdoo/lhm_animad_admin_html5/blob/402a448ccfa3a47110b1d316bd838c32e4d36ed3/src/behaviors/animad-form-behavior.html#L179

Suche für weitere Stellen

Wird die kicked: true Information wo benötigt?

Expected Behavior

Nur das Event für den reload wird versendet.

Actual Behavior

Das Event wird mit wahrscheinlich nicht benutzten Zusatzdaten versendet. Das detail: {kicked: true} wurde wahrscheinlich einfach aus der Polymer Doku mit kopiert.

Lösung:

Löschen :smile:

rowe42 commented 6 years ago

@peter-mueller Richtig. Kann man löschen. ABER: Wir machen ab sofort keine direkten Änderungen mehr im Zoo-Beispiel, ohne das auch im Generator anzupassen. Darum bitte stattdessen ein Issue im Gitlab aufmachen, gern auch selber fixen (in einem eigenen Branch basierend auf Branch polymer-ui2) und per MR an Boxi.

Ist übrigens im Security-Bereich auch so gemacht. Vielleicht magst du das im gleichen Zug auch gleich mitfixen (und natürlich testen, dass es danach noch wie vorher funktioniert).

Dieses Issue dann bitte wieder zumachen (gern mit Verweis auf das Gitlab-Issue).

Danke!

xdoo commented 6 years ago

This issue was moved to xdoo/move_issues#1