Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LCD Option not updating #14

Closed
MasterLH opened this issue Jul 28, 2021 · 7 comments · Fixed by #18
Closed

LCD Option not updating #14

MasterLH opened this issue Jul 28, 2021 · 7 comments · Fixed by #18

Comments

@MasterLH
Copy link

Hey, ich schreibe mal auf deutsch, hoffe, das ist okay.
Zunächst einmal vielen Dank für das coole Addon, ich bin begeistert von den Funktionen und kann das echt super auf meinem Smart Mirror gebrauchen.

Das Addon funktioniert an und für sich bei mir auch super, jedoch updatet es sich nicht, wenn ich die Display-Anzeige auf LCD-Modus umstelle...
Im "Details" Modus klappt alles einwandfrei und sonst habe ich nichts am Addon oder in der Config bearbeitet...

Habe bereits eine neuinstallation und mehrere Neustarts versucht, auch eine Anpassung des Update Intervalls hilft nicht...

Beim Start vom Magic Mirror zieht sich im LCD-Modus das Addon seine Daten, dann passiert aber leider nichts mehr, die Minuten werden nicht weniger und bereits abgefahrene Linien stehen weiterhin mit der Minutenzahl an, es wirkt wie eingefroren...

Nur wenn ich Magic Mirror neu starte zieht sich das Addon einmal kur die neuen Daten, dann friert es wieder ein.
Im Details Modus hingegen absolut keine Probleme, aber mir gefällt das LCD Aussehen so viel besser :)

Würde mich über einen Fix freuen, viele Grüße

@Klizzy Klizzy linked a pull request Feb 5, 2023 that will close this issue
@Klizzy Klizzy closed this as completed in #18 Feb 5, 2023
Klizzy added a commit that referenced this issue Feb 5, 2023
Bugfix for issue #14, refactored code and removed code smell
@Klizzy
Copy link
Owner

Klizzy commented Feb 5, 2023

Hallo @MasterLH ,

es ist zwar einiges an Zeit zwischen der Meldung und dem Bugfix vergangen, aber nun wird auch der displayType lcd korrekt aktualisiert. Danke für die Erstellung und Beschreibung der Issue! 👋🏼

@MasterLH
Copy link
Author

MasterLH commented Feb 6, 2023

Dankeschön fürs fixen. Leider kann ich nicht auf die neue Version updaten, da nach dem Update immer der folgende Fehler kommt:

[06.02.2023 14:25.02.168] [ERROR] Error: Cannot find module 'request'
Require stack:

  • /opt/magic_mirror/modules/MMM-Vrr/node_helper.js
  • /opt/magic_mirror/js/app.js
  • /opt/magic_mirror/js/electron.js
  • /opt/magic_mirror/node_modules/electron/dist/resources/default_app.asar/main.js
  • at Module._resolveFilename (node:internal/modules/cjs/loader:963:15)
    at n._resolveFilename (node:electron/js2c/browser_init:2:109416)
    at Module._resolveFilename (/opt/magic_mirror/node_modules/module-alias/index.js:49:29)
    at Module._load (node:internal/modules/cjs/loader:811:27)
    at f._load (node:electron/js2c/asar_bundle:2:13328)
    at Module.require (node:internal/modules/cjs/loader:1035:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object. (/opt/magic_mirror/modules/MMM-Vrr/node_helper.js:9:17)
    at Module._compile (node:internal/modules/cjs/loader:1141:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1196:10)

Auch ein Uninstall und Reinstall und das Einfügen der Beispiel-Konfiguration hat nicht geholfen

Klizzy added a commit that referenced this issue Feb 6, 2023
Klizzy added a commit that referenced this issue Feb 6, 2023
Klizzy added a commit that referenced this issue Feb 6, 2023
Closes #14 added node request module package.json, added keywords and commited package.lock
Klizzy added a commit that referenced this issue Feb 6, 2023
Klizzy added a commit that referenced this issue Feb 6, 2023
@Klizzy
Copy link
Owner

Klizzy commented Feb 6, 2023

@MasterLH

hmmm, bei dir scheint eine Abhängigkeit zu Fehlen, die eigentlich von der MagicMirror Software mitgeliefert wird.

Ich habe nochmal eine Anpassung vorgenommen und ausgespielt. Folge bitte erneut der Installationsanweisung, dann sollte sich der Fehler erledigt haben.

@Klizzy Klizzy reopened this Feb 6, 2023
@Klizzy
Copy link
Owner

Klizzy commented Feb 11, 2023

@MasterLH
Issue wird geschlossen. Falls noch Probleme auftreten, gerne wieder öffnen

@Klizzy Klizzy closed this as completed Feb 11, 2023
@MasterLH
Copy link
Author

Hey, jetzt funktioniert es tatsächlich!
Leider funktioniert im LCD-Modus die "Line" Auswahl nicht, bzw. es werden trotzdem alle Linien angezeigt. Ändert man dann nur den Display-Type und lässt sonst alle Parameter gleich, funktioniert der "Filter" line wieder...

Vielen lieben Dank schonmal. Hoffentlich klappt dann auch bald die Anwendung von Filtern im LCD-Modus

@Klizzy
Copy link
Owner

Klizzy commented Feb 13, 2023

@MasterLH ,

Ich habe die config parameter auch in der LCD Ansicht hinzugefügt und das readme aktualisiert. Dort ist nun zu sehen welche Konfigurationen von der LCD Ansicht nicht unterstützt werden.

Einmal das Plugin neu installieren, dann hast du auch die Änderungen ☺️

@MasterLH
Copy link
Author

@Klizzy

Hey, vielen Dank, nun funktioniert das Plugin tatsächlich wie es sein soll.
Wenn eines Tages noch die "withoutDestination" klappt wäre es natürlich super.

Bin sehr begeistert von dem Plugin - Dankeschön nochmal!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants