auch wenn es vielleicht nicht so schlimm war, trotzdem und da schliesse ich mich dem Dank von Harleh an, ein wirklich gut gelungenes Modul. Und da spielt die Größe oder der Aufwand gar keine Rolle. Es ist einfach eine gute Umsetzung des Themas. Vor allem die Google Maps Einbindung macht was her!
das Modul funktioniert, nur das einrichten ist ein bischen tricky (welche Instanz zuerst, etc.). Wenn ich die IO zuerst installiere, kommt der Fehler, das der Konfigurator nicht gefunden wurde. Den dann neu angelegt und siehe da, ich hatte zwei Konfigurator-Instanzen
Was aber noch praktisch wäre, wenn man im Modul die maximale Anzahl von Tankstellen im Umkreis eingrenzen könnte. Ich nutze das Ergebnis in einer HTML-Box in IPS-View und mir würden 5 reichen, anstelle von 25. Wir sind hier in Berlin etwas dichter belagert mit Tankstellen. Selbst bei einem Radius von 5km werden noch zuviele angezeigt und bei 3km keine mehr.
Ich bin immer dankbar, für solche Hinweise. Es ist auch das erste Module mit Konfigurator.
Neben dem von Dir genannten Fehler gibt es noch einen weiteren:
Wenn der Konfigurator in der Konsole geladen ist und man macht ein Update kommt en Fehler das das Instanz-Interface noch nicht verfügbar ist. Habe versucht die Datenaktualisierung des Konfigurators durch „HasActiceParent“ zu prüfen und damit diese nicht „ins Leere“ laufen zu lassen, ist aber nicht so richtig erfolgreich…
Das Andere: Ich habe bereits die notwendige Funktion zur Korrektur von Daten integriert, weiß aber noch nicht so genau wo ich diese unterbringen soll, aktuell schwebt mir das Konfigurationsformular zur einzelnen Tankstelle vor…
Das mit der maximale Anzahl werde ich versuchen in die Umkreissuche zu integrieren.
Wenn ich die IO zuerst installiere, kommt der Fehler, das der Konfigurator nicht gefunden wurde.
Ich finde das auch verwirrend mit den einzelnen Instanzen bei der Installation. Ich hatte mich auch schon öfters gefragt welche Instanz wohl zuerst installiert werden muss. Wobei mittlerweile gehe ich immer davon aus das zuerst der Konfigurator zu instanziieren ist. Was auch Sinn machen würde. Aber generell sollte man das auch von Symcon Seite aus noch einmal klarstellen. Ich meine welche Reihenfolge standardmäßig vorliegen sollte. Auf Dauer könnte dann vielleicht sogar die Darstellung der anderen Instanzen in der Instanzauswahl unterdrückt werden oder zumindest der Konfigurator hervorgehoben werden und dadurch die Sache für den User vereinfacht werden. Zumindest wenn es einen Konfigurator gibt.
Vielleicht könnte man auch dem Entwickler die Möglichkeit einräumen eine Instanz durch eine Kennung hervorzuheben. Dann könnte dieser in jedem Fall die erste zu erzeugende Instanz markieren.
Symcon gibt das schon vor.
Zumindest wenn man den Store nutzt
Zuerst Discovery, wenn es die nicht gibt Konfigurator. Den Rest weiß ich nicht aus dem Kopf
Michael
Danke, Joachim. Ich meinte eigentlich, das auf github in der readme.md einzutragen, damit es jederzeit offensichtlich und aktuell ist. In Threads geht so Information leider schnell unter.
…die Korrektur von Adressdaten habe ich mal in das Konfigurationsformular der Station eingebaut und getestet. Es gab keine Fehlermeldung, jedoch ist im Datenbestand von Tankerkoenig.de bisher keine Änderung sichtbar. Ich vermute, dass diese Änderungen manuell geprüft werden?
Andere als die Adressdaten habe ich nicht zur Änderung angeboten, ich vermute da mal das sich wohl kaum einer die Mühe geben wird, seine Konsole zu öffnen um den Spritpreis zu korrigieren…
Die UUID wird jetzt auf grundsätzliche Gültigkeit geprüft, der maximale Suchradius wird auf 25km begrenzt und sonstige kleine Dinge.
Wenn ich einen Wunsch äußern dürfte. Ich würde gerne eine Variable (Boolean) haben, die anzeigt ob offen oder geschlossen. Dann kann man sich nämlich die Anzeige der Tankstelle sparen
Danke für die schnelle Reaktion, war eine zeitlang offline;)
Ich meine eine Statusvariable, die dann ausgewertet werden kann, so wie der Spritpreis oder die letzte Aktualisierung. Die Angabe ist ja unter Tankstelle mitenthalten, lässt sich so aber nicht auswerten.