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

Update log.html #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update log.html #31

wants to merge 1 commit into from

Conversation

druckgott
Copy link

No description provided.

@madmartin
Copy link
Owner

Hallo @druckgott
Ich sehe diesen und diverse andere PullRequests.
In keinem steht eine Beschreibung, und so wie der Titel lautet wird immer eine Datei geändert.
Ohne da genau hinzusehen sage ich mal - so ist das mit dem "git" und denn PullRequests nicht gedacht.

Bitte den Grundsatz "eine logische Änderung -> ein Commit" befolgen. Ob eine oder mehrere Dateien dabei geändert werden ist nicht wichtig.
Und eine Beschreibung fehlt auch. "Update log.html" ist keine wirkliche Beschreibung, der Sinn des "Update" ist hier gefragt.

Dennoch danke fürs Engagement - ich bin gespannt worum sich die Neuerungen drehen.

@druckgott
Copy link
Author

druckgott commented Apr 14, 2020

Oh sorry, ich hatte es in die erste Datei geschrieben, die ich aber dann nicht gesendet habe, weil ich sie nochmal geändert habe.

Ich habe eine weitere Seite eingefügt, mit der über die URL Adresse oben jeweils der Command gesendet werden kann. Hintergrund ist, ich will über einen Cronjob oder Phyton Script z.B. mit Sonnenaufgang und Untergang meine Rollos steuern. Um jetzt nicht extra einen brocker einrichten zu müssen hab ich das reinprogrammiert.

In der Datei wurde nur ein Link zur "neuen" Seite eingefuegt.

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 this pull request may close these issues.

2 participants