Test/Support Adapter Skiinfo

[MCU]

@sigi234 Habe damit nichts zu tun.

[Homoran]

@oliverio Das müsste dann nach meinem bescheidenen Wissen ja eine dependency sein.
@sigi234 nutzt nach meinem Wissen Windows.
Ist da etwas anders als bei deinem Testsystem?

Ansonsten hab ich nur

"typescript": "~5.7.2"

auf die Schnelle im code gefunden was mir wegen der Tilde etwas seltsam vorkam, was nichts heißen muss

[OliverIO]

@homoran

irgendwas im io-package müsste da stören.

muss mal nochmal auf meinem testserver testen.
an dem adapter ist nix besonderes was das betriebssystem angeht.

[sigi234]

@homoran sagte in Test/Support Adapter Skiinfo:

@oliverio Das müsste dann nach meinem bescheidenen Wissen ja eine dependency sein.

Ja.

[Homoran]

@sigi234 was steht denn da sonst üblicherweise

wenns die io-package ist, steht da nur der Controller drin

der ist Pos1
Pos 3 ist node, bliebe noch npm

[mcm1957]

@oliverio
Ich tippe mal dass die LEERE dependency da stört:

    "globalDependencies": [
      {}
    ]

Also nur

    "globalDependencies": [
    ]

Oder auch global dependency ganz weglassen (ja ich weiß, ja beschwert sich der repochecker zur Zeit, aber das steht auf der Bugliste).

Kannst du das mal testen? Wenn dem aber so ist wär das m.E. ein Bug für den js-controller (oder ev. admin)

EDIT:
@Homoran
Nö - da steht auch {} was nichts anderes ist als 'undefined' mit beliebiger Version :-). Und genau 'undefined' ist nicht installiert

[sigi234]

@mcu sagte in Test/Support Adapter Skiinfo:

@sigi234 Habe damit nichts zu tun.

Sorry, hab dich verwechselt.

[OliverIO]

@mcm1957 sagte in Test/Support Adapter Skiinfo:

globalDependencies

ja, globalDependency war es wahrscheinlich.
Allerdings kann ich das aktuell nicht richtig testen, da aus irgendeinem Grund im Rahmen des Installationsdialogs des admins nicht gegen die neuen Informationen, sondern denen aus dem iobroker-Repository geprüft wird, wo noch das leere Objekt drin steht.
Ich versuchte von NPM die Version 0.41 zu installieren.

Wenn ich dann an dieser Stelle reinschaue, dann ist im Objekt adapter.globalDependency immer noch das leere Objekt drin und in adapter.version steht 0.4.0 drin.

https://github.com/ioBroker/ioBroker.admin/blob/master/packages/admin/src-admin/src/components/Adapters/AdapterInstallDialog.tsx#L297

Der Sinn erschließt sich mir hier nicht. Ich gehe davon aus, das man dann warten muss bis das iobroker beta repo aktualisiert wurde.
Das könnte aber in Zukunft öfters Probleme machen, wenn jemand eine neuere Version von npm installieren will um etwas zu testen und das beta repo noch nicht aktualisiert wurde.

Ich gehe aber davon aus, das es mit der version 0.4.1 gefixt ist.

[mcm1957]

@oliverio said in Test/Support Adapter Skiinfo:

ja, globalDependency war es wahrscheinlich.
Allerdings kann ich das aktuell nicht richtig testen, da aus irgendeinem Grund im Rahmen des Installationsdialogs des admins nicht gegen die neuen Informationen, sondern denen aus dem iobroker-Repository geprüft wird, wo noch das leere Objekt drin steht.

Ja das ist das implementierte / gewünschte Verhalten. Admin / js-controller benutzt die Informationen aus dem Repository da der der npm Inhalt ja erst nach der Installation zur Verfügung steht, die Prüfung aber vor der Installation erfolgen soll. Ja, kann man bestimmt auch andere Modelle durchdenken. Probleme diesbezüglich sind (mir) nicht bekannt. Allfällige Änderungswünsche / Vorschläge gerne als Issue zum js-controller erfassen oder in Telegramm mit @Apollon77 / Bluefox / Foxriver diskutieren.

Ich habe einen Repobuild getriggert. Wenn du magst kannst du in ca 20-25 Minuten testen. Dann sollte ein aktualisisertes Repo verfügbar sein.

EDIT: Für Entwicklungszwecke kann der Entwickler den Repoinhalt in seinem Cache (system.xxx Objekt) ggF manipulieren,

[OliverIO]

@mcm1957

der Zeithorizont ist auch relativ kurz (zwischen publish und repoaktualisierung) bei dem das Problem auftritt.
Es überrascht halt, da man ja extra "installieren aus npm" angewählt hat.
dann dürfte man hier eigentlich nur installieren aus github machen, wo das theoretisch nicht gegen das repo geprüft wird.

danke fürs aktualisieren. ich schaue gleich mal wenn es dann da ist

[mcm1957]

@oliverio
Installieren aus Github prüft gar nix. Da kannst das auch nicht testen

Vergiss nicht auf "nach updates suchen" zu klicken damit der lokale repocache aktualisisert wird

[OliverIO]

Neue Version 0.4.1

Das Installationsproblem wurde mit dieser Version behoben

[sigi234]

@oliverio sagte in Test/Support Adapter Skiinfo:

Neue Version 0.4.1

Das Installationsproblem wurde mit dieser Version behoben

Danke, läuft

[bommel_030]

@oliverio sagte in Test/Support Adapter Skiinfo:

Neue Version 0.4.1

Das Installationsproblem wurde mit dieser Version behoben

Installation problemlos, bei der Auswahl braucht man tatsächlich etwas Geduld, aber Favoriten an- und abwählen funktioniert auch einwandfrei.
Werden die Spaltenüberschriften automatisch übersetzt, oder hast du das fix eingetragen?
Bereich würde ich da eher mit Gebiet bevorzugen. "Datum von" ist mit Update vielleicht besser dran. Der "Aufzug" statt "Lift" triggert mich auf jeden Fall

[OliverIO]

@bommel_030

Was meinst du mit der Auswahl und Geduld?
Ist das Favoriten Icon zu klein?

Die Übersetzung kann ich noch mal checken.
Ist alles in Translation Files drin?

[bommel_030]

@oliverio
Nein, das Favoriten Icon passt. War nur ne Rückmeldung, dass das an- und abwählen einwandfrei funktioniert.
Geduld daher, wenn ich ein Land auswähle ist das erste Bundesland/Kanton vorausgewählt und die darin enthaltenen Skigebiete werden angezeigt.
Wähle ich das nächste Bundesland/Kanton aus ist die Liste mit Gebieten erstmal leer. Es dauert eine Weile bis diese dann aktualisiert wird.

[OliverIO]

@bommel_030

Ah, ja.
Das wird vorgeladen. Bei Auswahl wird nach geladen. Allerdings musste ich da einiges an Verzögerung einbauen, dass der Web Server überhaupt antwortet.
Das steht Eingangs oben mit dabei.
Hab leider noch nicht herausgefunden an was das liegt, dass der Web Server weiß, dass hier Node und nicht der Browser fragt.
Im Browser funktioniert schnelle Abfrage hintereinander einwandfrei

Danke fürs testen

[OliverIO]

Neue Version 0.5.0

• Eine Suchmöglichkeit für Regionen/Skigebiete wurde eingebaut.
• Die Übersetzung wurde vereinzelt verbessert.

[OliverIO]

Auch wenn die Temperaturen dazu gar nicht passen.
Der Adapter befindet sich nun mit der Verion 0.5.0 im stable

[sigi234]

@oliverio sagte in Test/Support Adapter Skiinfo:

Auch wenn die Temperaturen dazu gar nicht passen.
Der Adapter befindet sich nun mit der Verion 0.5.0 im stable

Cool, Super wäre es wenn es so einen Adapter auch für Wassertemperaturen geben könnte.