0183-WebUI-ImprovedFirmwareupdateDialog

[Baxxy13]

Description

Modified "new DeviceFirmware PopUp".

Related Issue

#2396 (comment)

Types of changes

• Docs change / refactoring / dependency upgrade
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Alternate Designs

Possible Drawbacks

Nothing known / found so far.

Verification Process

I'm using this patch since some months on different RaspberryMatic-Systems. All is working as expected.

Release Notes

Reworked colorized design which groups same devices and count them.
Small help and shortcut for updating implemented.

Contributing checklist

• My code follows the code style of this project.
• I have read the CONTRIBUTING and LICENSE document.
• I fully agree to distribute my changes under Apache 2.0 license.

[Baxxy13]

Aktuell sieht das so aus:
(ohne die gelbe Markierung)

[jens-maus]

Wie du siehst lässt sich aktuell der neue WebUI Patch nicht anwenden (weil in einem ähnlichen Bereich schon in vorherigen WebUI patches Änderungen vorgenommen wurden). Siehe

https://github.com/jens-maus/RaspberryMatic/actions/runs/7009367224/job/19067636364?pr=2532#step:13:7108

Müsstest du also noch einmal einarbeiten.

[Baxxy13]

Ok.
Das ist für mich Neuland. Wie gehe ich das jetzt an?

[IT-VBFK]

Wenn ich bisherige Hilfen (von Jens und Jerome) richtig verstehe, dann musst du als webui.js.orig die webui.js aus dem letzten Patch (zB Patch 173) nehmen, der in diesem Bereich was verändert (und dann darauf aufbauend deine Änderungen machen)

[jens-maus]

Wenn ich bisherige Hilfen (von Jens und Jerome) richtig verstehe, dann musst du als webui.js.orig die webui.js aus dem letzten Patch (zB Patch 173) nehmen, der in diesem Bereich was verändert (und dann darauf aufbauend deine Änderungen machen)

Das wäre ein Ansatz, ja. Und vermutlich auch der einfachste für @Baxxy13.
D.h. einfach mal schrittweise durch alle WebUI patch nummern kleiner als die neue durchgehen und in dem Patch wo die webui.js das letzte mal abgeändert wurde diese dann im neuen Patch als webui.js.orig datei nutzen und darauf basierend dann die Anpassungen erneut umsetzen.

Alternativ kann man auch einfach schauen welcher "Hunk" des patch files sich nicht anwenden lässt bzw. welcher zu einem fehler führt und dann die *.rej Datei in dem jeweiligen Verzeichnis anschauen und die notwendigen Änderungen dann in der *.orig Datei einpflegen damit das patch file sich wieder anwenden lässt.

Hilfreich bei der ganze Sache ist auch immer make check im top-level verzeichnis des repositories auszuführen. Denn dieser Befehl macht genau das was die CI/CD Engine hier in GitHub auch macht, es prüft ob alle patches sich korrekt wenden lassen oder ob irgendwo in der Patch chain ein fehler existiert.

[Baxxy13]

Ah ja, sowas hatte ich mir gestern schon gedacht und das mal mit der webui.js aus 0127 probiert.
Das klappte aber ebenso nicht, ich muss also eine "spätere Version" nehmen.

Wenn ich das richtig gesehen habe ist 0181 der aktuell "letzte" Patch der die webui.js anfasst.
Von dem würde ich jetzt die webui.js als webui.js.orig nehmen.

Das mit den .rej Files klingt mir erstmal zu kompliziert, ich gehe den einfachen Weg.

[jens-maus]

Probier es. Zu beachten gilt nur, das du sicherstellst nicht einfach copy&paste zu machen, sondern gerade wenn in der exakt selben Zeile in vorherigen WebUI patches änderungen vorgenommen wurden du diese änderungen nicht mit dem neuen Patch quasi rückgängig machst oder Funktionen damit veränderst...

[Baxxy13]

Na hoffentlich steht in den nächsten Release-Notes nicht "Baxxy hat die RaspberyMatic kaputt gemacht". 😉

Wie finde ich denn den Patch der im betreffenden Bereich Änderungen gemacht hat?

[jens-maus]

Wie finde ich denn den Patch der im betreffenden Bereich Änderungen gemacht hat?

Du hast ihn ja anscheinend gefunden, denn der CI Build Check lief ja jetzt nach deinen letzten Anpassungen sauber durch.. sehr gut :)