j-hiller / MMM-AC-aseag

Magic Mirror 2 module to show departures of ASEAG busses in Aachen, Germany
MIT License
2 stars 2 forks source link

Umbau zu neuer AVV-API #3

Closed joshua-martius closed 2 years ago

joshua-martius commented 2 years ago

Hi,

hier meine Version, welche bei mir funktioniert. Die Access-ID muss wie gesagt bei der AVV direkt angefordert werden, über einen Antrag. Der gute Mann von der AVV ist allerdings super nett und flott im Antworten.

Die StopID ist hier jetzt nicht mehr optional sondern required, da ich diese Version ursprünglich nicht veröffentlichen wollte und die ID meiner Station wusste. Wenn ich Zeit finde, würde ich das eventuell noch einbauen. Da ich mich in diesem Semester jedoch mit Mathematischer Logik rumschlagen muss, denke ich nicht dass ich dafür noch Zeit haben werde.

Beste Grüße

j-hiller commented 2 years ago

Ich bin noch nicht dazu gekommen, einen API-Key zu beantragen. Habe das eben nachgeholt und werde dann deinen Code testen. Sorry für die Verzögerung.

j-hiller commented 2 years ago

Ich habe noch einen Fehler in der Abfrage der Busse gefunden, siehe mein Kommentar im Code. Ansonsten scheint die Aktualisierung nicht zu funktionieren, bin mir aber nicht sicher, ob das an meinem Test-Setup liegt

joshua-martius commented 2 years ago

Hi,

ich konnte keine Kommentare im Code finden. Das ganze funktioniert auch bei mir soweit... Kannst du nochmal schreiben wo das Problem liegt?

j-hiller commented 2 years ago

In node_helper.js wo du die einzelnen Abfahrten abfragst. Bei mir kommt da eben ein Fehler, da es e.Product.name nicht gibt. Meiner Meinung nach müsste es entweder e.Product[0].name oder e.ProductAtStop.name sein, auch nachdem ich mir angeschaut habe was zurück geliefert wird von der API.

Aber komisch, dass es bei dir dann einfach so funktioniert.

joshua-martius commented 2 years ago

Tatsächlich hat es bei mir auch nicht mehr funktioniert. Ich habe aber auch festgestellt dass die API vor kurzem für einige Tage nicht/schwer erreichbar war. Vielleicht gab es ein Update? Habe das jetzt angepasst.

Voltfest commented 2 years ago

Hi,

erstmal danke, top Arbeit, habs direkt in Betrieb genommen! Ein paar Sachen sind mir noch aufgefallen bzw habe ich bei mir noch angepasst, siehe dazu den angehangenen Screenshot (da ich kaum Programmiererfahrung habe allerdings hardgecoded und wahrscheinlich unprofessionell)

2022-07-19-215011_1080x1920_scrot

joshua-martius commented 2 years ago

Hi,

sorry dafür, dass das jetzt doch nicht funktioniert. Ich weiß nicht wie das sein kann, ich hatte es eigentlich zu Genüge getestet. Ich werde mir zeitnah ne Lösung für das Grundproblem des Updatens überlegen. Ich kann mir aber keine Zeit nehmen die angesprochenen Komfortfeatures umzusetzen.

Voltfest commented 2 years ago

Kein Problem, ich habe es bei mir ja entsprechend umsetzen können. Die restlichen Vorschläge wären, wie du schon sagst, nice to have für weitere Nutzer, aber nicht lebenswichtig ;)

Voltfest commented 2 years ago

@j-hiller funktioniert es denn bei dir noch?

j-hiller commented 2 years ago

@Voltfest sorry für die späte Antwort... Ich glaube ich habe das Problem gefunden, prüfe doch mal bitte #4 Zumindest bei mir funktioniert es damit.

Voltfest commented 2 years ago

Kein Ding, danke dass Du dich drum kümmerst. Ich kriege leider gar keine Verbindungen mehr. API & Key funktioniert noch wenn ich es im Browser versuche, aber im Modul steht nur "Loading connections ...". grafik Das Problem müsste schon länger bestehen, ohne dass ich was am Code verändert habe. Mit der alten Version wurden die Verbindungen ja zu Beginn einmal korrekt geladen, bei den Letzten Spiegel-Neustarts sah das Modul immer aus wie im Screenshot. Hast Du ne Idee, woran das liegen könnte?

j-hiller commented 2 years ago

So ganz nachvollziehen kann ich es nicht, nein, da selbst nach mehreren Neustarts und mit Misereor es bei mir die Verbindungen lädt. Nicht mal mit falscher accessId bleibt er bei mir auf Loading connections ... hängen. Kannst du es vielleicht nochmal mit einer ganz frischen config testen?