LukeSkywalker92 / TeleFrame

TeleFrame - a digital picture frame for telegram
MIT License
93 stars 28 forks source link

WIP: Play pause back forward #14

Closed s00500 closed 5 years ago

s00500 commented 5 years ago

Ok ich glaube es ist cool eine simple 4 button Steuerung zu haben...

Die letzte frage die sich mir noch stellt ist ob ich globale shortcuts in electron verwenden soll, oder noch electron-localshortcuts einbauen soll....

Was meinst du ?

s00500 commented 5 years ago

btw: this pr includes my previous #13, so pls merge #13 first

LukeSkywalker92 commented 5 years ago

Was spricht denn gegen globale shortcuts? Bzw. was spricht für electron-localshortcuts und wieviel aufwand würde das bedeuten? Was mir wichtig wäre, ist eine Variable in der config, mit der man die Steuerung ein und aus schalten kann.

LukeSkywalker92 commented 5 years ago

Ein bisschen was off-topic: Du müsstest mal wieder das Zertifikat von deiner website lbsfilm.at erneuern. Hast du root-zugriff auf den server? Dann kann man das auch ganz einfach automatisieren. Ich mach das mit einem Skript in cron.monthly so:

SHELL=/bin/sh
PATH=/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin

certbot -q renew --pre-hook "systemctl stop apache2" --post-hook "systemctl start apache2"

telegram-notify --success --text "Certbot certificates renewed."
s00500 commented 5 years ago

Hahaha, aber warum ? cert

Das rennt bei mir alles automatisiert ;-)

LukeSkywalker92 commented 5 years ago

Interessant, jetzt ist's bei mir auch bis zum 2. Juni gültig. Als ich vor 10 Minuten über dein GitHub Profil die Seite aufrufen wollte, hat mir mein Browser gesagt, dass das Zertifikat abgelaufen ist. Musste dann eine Ausnahme einrichten um die Seite zu besuchen.

Vielleicht liegts auch an meinem Browser 🤣

s00500 commented 5 years ago

sehr strange =P

@topic: nein, eigentlich spricht nix gegen globale shortcuts... ich hab das ding auch bis jetz noch auf keinem raspi laufen gelassen, sondern nur local, da is es dann ein wenig nervig weil unter macOS greift es dann IMMER auf den shortcut zu, auch wenn die app im hintergrund ist, aber nicht weiter schlimm...

config options zum ändern der tasten gibts schon, ich könnte es noch so machen dass man das keys object einfach null setzen kann, dann gibts keine kontrolle

LukeSkywalker92 commented 5 years ago

Das hört sich vernünftig an. Ich hatte noch keine Zeit den ganze Code durchzugehen. Vielleicht hast du das was jetzt kommt schon so implementiert.

Ich könnte mir vorstellen, dass es in der Zukunft auch interessant werden könnte die Steuerung über physische Schalter/Knöpfe/etc. am rpi zu realisieren. Es wäre also top, wenn das später einfach implementiert werden kann.

LukeSkywalker92 commented 5 years ago

6E96A917-AC14-4CDE-A609-EC0B4BD3E679

Irgendwas läuft da schief.

s00500 commented 5 years ago

stimmt das datum in deinem handy? bei mir passt alles, ev mal wieder ein ios update machen?

LukeSkywalker92 commented 5 years ago

Screenshot from 2019-04-27 15-33-59 iOS ist aktuell und datum stimmt auch. Tritt auch in Firefox auf arch linux auf.

s00500 commented 5 years ago

 Haha well das cert hat auch keinen alias für www =P

s00500 commented 5 years ago

habs im profil korrigiert, thx

LukeSkywalker92 commented 5 years ago

😂

Nutzt du Apache? Dann kannst du www einfach auf ohne www umleiten.

s00500 commented 5 years ago

nginx, allerdings mit nem controlpanel, ja sollt ich mal in die templates adden, anyway, offtopic =P

s00500 commented 5 years ago

Can we merge this?

LukeSkywalker92 commented 5 years ago

I had no time to test it yet. Is everything working as expected?

s00500 commented 5 years ago

Yes it is

On Saturday, May 04, 2019 at 9:07 AM, LukeSkywalker92 <notifications@github.com (mailto:notifications@github.com)> wrote:

I had no time to test it yet. Is everything working as expected?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub (https://github.com/LukeSkywalker92/TeleFrame/pull/14#issuecomment-489301963), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAL36L433TR7LSK22KZC7HTPTUY4HANCNFSM4HIWYDBQ).

LukeSkywalker92 commented 5 years ago

Can you please add the config parameters for the keys to the README? Then I'm going to merge it.

s00500 commented 5 years ago

Check the readme on my branch, its already there ;-)

LukeSkywalker92 commented 5 years ago

I missed it somehow, while reading through the changes :)