rowe42 / lhm_animad_admin_html5

0 stars 6 forks source link

Experimental: Inject Maven project version into footer #252

Closed eidottermihi closed 6 years ago

eidottermihi commented 6 years ago

Ich stelle mal ein PR, um mit euch zu klären, wie wir folgende Anforderung umsetzen können:

Als Tester möchte ich auf der Oberfläche der Anwendung erkennen, in welcher Version die Anwendung gerade vorliegt/deployed ist.

Ich habe dazu eine Komponente animad-footer angelegt, die über gulp und Maven die aktuell gebaute Version (= Mavens project.version) zur Build-Zeit injiziert bekommt.

Evtl. besteht über diesen Weg auch die Möglichkeit, z.B. die Backend-URL während der Build-Zeit zu setzen (see #189).

Nachteil dieser Lösung ist, dass wir den Build über Gulp (oder andere JS Build Tools) bewerkstelligen müssen und damit an dieser Stelle die Komplexität steigt. Aber ich habe keine Möglichkeit gefunden, diese Anforderung nur mit der Polymer CLI (polymer build) hinzubekommen.

@xdoo / @rowe42 Sowas in diese Richtung werden wir für BeZweck benötigen.

xdoo commented 6 years ago

@eidottermihi Hilfe und gute Ideen sind immer herzlich willkommen :) ABER: Bitte das nächste mal die Sachen vorher in ein Issue schreiben, damit alle Beteiligten die Möglichkeit haben vorher darüber zu diskutieren.

Den PR merge ich.

rowe42 commented 6 years ago

@xdoo da du das gleich gemerged hast: hast du das getestet? Ist das Ergebnis identisch mit polymer build? minifiziert und transkribiert nach ES5? siehst du irgendwelche Nachteile?

xdoo commented 6 years ago

@rowe42 hat Recht. Das war etwas vorschnell. Der Travis Build ist nach dem Merge fehl geschlagen und ich habe ihn nochmal reverted. @eidottermihi bitte ein Issue aufmachen, indem wir die von @rowe42 gestellten Fragen beantworten und dann können wir es nochmal probieren. Bitte auch nochmal die Codebasis des Forks erneuern.