[seaspotter]

Will solve: #3687

UI: openWB/openwb-ui-settings#1020

[AlexanderHa98]

Hallo,
bitte ergänze den neuen Parameter auch in der update_config.py da hier ein bestehendes System erweitert wurde.

[seaspotter]

Hallo, bitte ergänze den neuen Parameter auch in der update_config.py da hier ein bestehendes System erweitert wurde.

@AlexanderHa98 das stimmt so nicht, denn in der config.py wird es mit dem PR gesetzt auf jq_frequency: Optional[str] = None):
Da es vorher keine Frequenz gab, wird also die bestehende Config automatisch ein None für diesen Parameter bekommen und es ist keine update_config nötig. Das ist ja nur der Fall wenn ich einen bereits bestehenden Paramter ändere oder umleite auf einen anderen Wert/ein anderes Modul. User die nun eine Frequenz angeben wollen, weil sie verfügbar ist, müssen also ihre Config updaten/ergänzen.

[AlexanderHa98]

Ja, hier geht es aber um die Daten im Broker.

Wurde ein Gerät mit einer älteren Version erstellt, in der es den Frequenz-Parameter noch nicht gab, dann existiert dieser Parameter auch nicht im Broker und wird entsprechend auch nicht übertragen. Also auch nicht mit dem Wert None.

Deshalb muss update_config.py angepasst werden. Dort werden genau solche Migrationsfälle behandelt, damit bestehende Geräte bzw. Komponenten nachträglich den neuen Parameter im Broker erhalten und keine inkonsistenten Zustände entstehen.

Bei Geräten, die bereits mit der neuen Version erstellt werden, tritt dieses Problem nicht auf. Die Anpassung dient ausschließlich dazu, bestehende Konfigurationen auf den neuen Stand zu bringen.

[seaspotter]

Erledigt