VIS1 - Problem mit justgage
-
@mcm57
Guten Morgen. Update auf 2.0.2 gemacht, aber leider immer noch keine Titel da. De- und Neuinstallation vom Adapter auch ohne Erfolg. Soll ich VIS mal komplett runterschmeißen?
-
Sorry, ich habe zu dem Problem derzeit keine Idee. Da ich kein VIS Spezialist bin, möchte ich nicht zu einer Deinstallation raten - ich kenne schlicht und einfach die Auswirkungen / Nebeneffekte nicht.
@apollon77 @foxriver76 @Bluefox
Wer kann sich das ansehen?McM57
-
@mcm57 ich fürchte nur @Bluefox
-
Kenn mich zwar mit frontends fast nicht aus. Aber was mir noch eingefallen ist:
Schau bitte mal ob du in der Browserconsole irgendeine Fehlermeldung siehts. Fürchte zwar dass das zumindest mir nicht weietrhilft - aber ev. ists ja was ganz eindeutiges ...
mcm1957
-
ich habe mal reingeschaut
die widgets nutzen hier eine bibliothek justgage (schöner tippfehler, sollte doch eigentlich justgauge heißen)
die bibliothek ist im adapter direkt mit enthalten.
lustigerweise wird im widget an die bibliothek zwar ein titel attribut übergeben, aber in der bibliothek wird das nicht genutztnach ein wenig suchen habe ich den folgenden commit von 2016 gefunden
https://github.com/toorshia/justgage/commit/bcf29f18c8b6f3e5ffb60c39d6d8e77ffbf993cc
darin wurde title komplett aus der bibliothek entfernt. es gibt nur noch label, welches unterhalb der gauge angezeigt wird.daher als workaround einfach ein eigenes label drüber oder drauf platzieren
die verschiedenen titel attribute können aus dem widget entfernt werden -
Herzlichen DANK.
Dann kommt dieses "Feature" wohl durch die im Zuge der Übersiedlung und Neuerstellung der Release erfolgte Aktualisierung der dependencies zu Stande.Jetzt brächte man nur noch jemand der weiß wie man ein Widget anpasst
-
@oliverio @mcm57
Danke für die Info. Wann tritt denn dieser Fehler ans Tageslicht? Ich hab mein System in letzter Zeit mehrfach neu aufgesetzt, und da war das nach dem restore nie ein Problem.
-
@1topf
das weiß ich nicht. ich nutze die widgets nicht
ich habe nur reingeschaut in den source und das was man bei title eintippt
kann nie ankommen -
Title wurde im Original ca vor 4 Jahren entfernt.
In ioBroker.vis-justgage ist ein Kopie des Originals eingebunden (als source)In v1.0.1 ist die Version 1.3.1 des originals (zumindest lt. Kommentar im Source). Diese Version kennt keinen Title mehr.
In v1.0.2 ist eine vermutlich ältere Version des Originals enthalten (ohne Versionsangabe im source). Diese enthält Title Code.
In V2.x.x ist wieder die Version 1.3.1 wie in v1.0.1 enthalten. Damit gibt es Title nicht mehr.
Problem ist, dass die Version 1.0.2 auf Github scheinbar gar nie erstellt wurde. Hier dürfte 'jemand' einen anderen Fork auf npm gepushed haben. Diese Änderungen sind dann natürlcih nicht im Original enthalten sofern kein PR erstellt wurde.
Da ich zu wenig Ahnung von vis / widgets habe kann ich hier nicht beurteilen, ob ein Downgrade des betroffenen Files sinnvoll ist.
Generell stellt sich aber auch die Frage ob dieses Widget mit vis-2 kompatibel ist oder ob es gemeinsam mit vis (v1) ein Ablaiufdatum hat da zumindest derzeit kein mir bekannter intensiver Wartungssupport für vis (1) besteht.
-
@mcm57
hier mal als alternative:
-
entfernen des title attributes
https://github.com/iobroker-community-adapters/ioBroker.vis-justgage/pull/57 -
ältester tag bei dem title noch enthalten ist:
https://github.com/toorshia/justgage/tree/1.2.5
ob der rest da noch kompatibel ist, keine ahnung
der allererste tag auf github ist v1.1.0
da nur als bower packet ohne npm
https://github.com/toorshia/justgage/tree/v1.1.0 -
-
@mcm57 sagte in VIS1 - Problem mit justgage:
Generell stellt sich aber auch die Frage ob dieses Widget mit vis-2 kompatibel ist oder ob es gemeinsam mit vis (v1) ein Ablaiufdatum hat da zumindest derzeit kein mir bekannter intensiver Wartungssupport für vis (1) besteht.
ich würde sowas aktuell nur nach best effort machen.
so wie es aussieht, läuft vis-2 aus meiner sicht noch nicht ganz so rund, als das man das als vollwertiger ersatz für vis-1 anbieten kann.
zumindest müssen dann einige dann ihre visualisierungen überarbeiten oder komplett neu machen.
allerdings fehlen dann noch einige widgets, die nur mit vis1 laufenidittmar hat letzt eine anfrage bei 2 von meinen adapter gehabt, das ich die dependency zu vis 1 entfernen soll, da es dann ja auch immer installiert.
er wollte in einem eurer dev meetings mal klären wie den die empfohlene adapter strategie sein soll:
serverlogik und vis-widgets in separate adapter, sovie auch vis1 und vis2 widgets in separate adapter? dann könnte man die dependencys entsprechend setzen
führt halt dazu, das wenn jemand noch vis1 und vis2 supporten möchte, das dann 3 adapter existieren (serverlogik,vis1 widgets, vis2 widgets)
ein namensschema wäre auch nicht schlecht, das man einheitlich erkennen kann was was ist
separate adapter hätte dann auch den charme, das für andere visualisierungen ebenfalls was erstellt werden kann. multipliziert halt die anzahl der adapter -
Vorsicht: vis-justgage benutzt offenbar eine Kopie des von dir referenzierten codes. Es gibt keine Abhängigkeit zum orinalen Paket.
Damit gobts noch Variante 3
Der Ersteller der 1.0.2 gat noch den fork ... -
@mcm57
warum ist dann diese datei ersetzt worden? -
@oliverio
Was die Abhängigkeiten betrifftWenn der Adapter auch ohne vis läuft braucht es die Abhängigkeit nicht. Die Widgets liegen dann galt ungenutzt rum.
-
@mcm57
bei meinen adaptern sind die widgets meist notwendig.
natürlich kann sich jemand da auch selbst was bauen,
aber bei rssfeed kann niemand so ohne weiteres nur den serveranteil nutzen -
@mcm57 sagte in VIS1 - Problem mit justgage:
Der Ersteller der 1.0.2 gat noch den fork
das scheint der original author zu sein
https://github.com/Pmant?tab=repositoriesaber leider kein repo mehr dazu.
ich persönlich würde das titel attribut herausnehmen.
das hat ja dann, seit der adapter bei den community adaptern ist nie wieder funktioniert (vor 4 Jahren), das original ist verloren, keiner hat lust das mit einer alten version wieder kompatibel zu machen. workaraound ist einfach: ein label widget drauf zu packen -
@oliverio said in VIS1 - Problem mit justgage:
@mcm57
bei meinen adaptern sind die widgets meist notwendig.
natürlich kann sich jemand da auch selbst was bauen,
aber bei rssfeed kann niemand so ohne weiteres nur den serveranteil nutzenOK, das mach bei rssfeed wahrscheinlich auch 'nen Sinn.
Bei exchangerates ist es aber durchaus sinnvoll nur die Daten in den states bereitzustellen. Ob der User die dann mit vis oder sonstwie anzeigen will, ist seine Sache. Das Adapetr kann ja die widgets jederzeit installieren - auch wenn vis nicht installiert ist. Oder? Hier erscheint es mir auch weniger sinnvoll vis zu erzwingen.
Aber das ist ein anderes Thema, das kann man beim Meeting besprechen - dazu gibts eh ein Topic.
-
@oliverio said in VIS1 - Problem mit justgage:
@mcm57 sagte in VIS1 - Problem mit justgage:
Der Ersteller der 1.0.2 gat noch den fork
das scheint der original author zu sein
https://github.com/Pmant?tab=repositoriesaber leider kein repo mehr dazu.
ich persönlich würde das titel attribut herausnehmen.
das hat ja dann, seit der adapter bei den community adaptern ist nie wieder funktioniert (vor 4 Jahren), das original ist verloren, keiner hat lust das mit einer alten version wieder kompatibel zu machen. workaraound ist einfach: ein label widget drauf zu packenDas pmant Repo ist ident dem community repo, das wurde verschoben.
Die 1.0.2 wurde direkt nach npm deployed - Namen will ich hier bewußt nicht nennen da ich nicht 200% sicher bin und es ja gute gründe für diese Vorgangsweise gegeben haben kann. Ich hab denjenigen schon angeschrieben und warte auch Rückmeldung. -
Wenn zumindest einer von euch bereit ist etwas zu testen, dann kann ich versuchen die 1.0.2 Änderungen in 2.x.x zu mergen und damit Topic wieder zum Leben zu erwecken.
Es wäre nur notwendig, dass jemand das widget ernsthaft durchtestet. Da ich mich mit vis kaum auskenne, kann ich das leider nicht tun. Und ich will den IST Stand nicht verschlimmbessern.
Bitte daher kurz Bescheid geben. Wenn ein tester verfügbar ist schau ich dass ich heute eine Testevrsion zusammenstelle. Hoff es geht sich zeitlich aus. (Muss grad mal Bay suchen gehn - der krabbelt schon schnelle als ich hier was schreiben kann
) -
@mcm57
ich kann da reinschauen, auch wenn ich normalerweise das widget gar nicht nutze
