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

Alarmierungen in ICS-Kalender übernehmen #462

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

nobbie2009
Copy link

Gleich vorweg, es ist mein erstes Plugin, es ist mein erster PullRequest .. bitte um Nachsicht ;)

Dieses Plugin, macht wie der Titel schon sagt nichts anderes als eine Alarmierung in eine ICS-File zu schreiben.
Benötigt werden hierfür noch folgende Projekte:

icalendar https://pypi.org/project/icalendar/
dateutil https://github.com/dateutil/dateutil/
pytz https://pypi.org/project/pytz/

@Schrolli91 Schrolli91 changed the title Alarmierungen in ICS-Kalender übernehmen (2. Versuch) Alarmierungen in ICS-Kalender übernehmen Jul 20, 2020
@flothi
Copy link
Collaborator

flothi commented Jul 20, 2020

Vorschlag: Sinnig wäre es, die Funktion des "in den Kalender schreibens" unabhängig vom Alarmtyp zu machen und lediglich die Summary (bei ZVEI) bzw zusätzl. ggf die Description (bei POCSAG) anzupassen. Das sollte ggf in der Konfig schon vorbereitet werden (über die bekannten Wildcards)

Frage am Rande:
Welcher Kalenderprovider liest denn die ics aus? Oder wird die händisch dann in Outlook etc integriert?

@nobbie2009
Copy link
Author

Man kann die Kalenderdatei wie eine normale ICal-Datei einbinden, zum Beispiel in Outlook.

Wegen Pocsac wie gesagt, habe ich keine Erfahrungswerte wie und welche Daten wichtig wären. Aber nur zu ;)

Adding support for POCSAG by globalizing functionality
@flothi
Copy link
Collaborator

flothi commented Jul 21, 2020

Ich war mal so frei und habe es ein wenig umgestrickt ;)

Copy link
Owner

@Schrolli91 Schrolli91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nur Kleinigkeiten + Changelog - ansonsten LGTM

@@ -186,7 +186,7 @@ fhemCmd = 0

# for developing - template-module
template = 0

2calendar = 1
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das sollte aber noch über den template Eintrag

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@flothi wieso resolved?

Erstellt eine Icalendar (ICS)-Datei, diese kann in Kalender wie Outlook etc. importiert werden.
Aktuell wird jeder Alarm der via Regex dieses Plugin anspricht, in eine "Gesamtdatei" geschrieben. Denkbar wäre aber auch
für jeden Filter eine seperate Datei anzulegen.
Auch wird aktuell nur ZVEI, aus mangel an Erfahrung, ausgewertet.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der Kommentar wäre mit der Anpassung von @flothi überflüssig bzw. nur noch FMS ist ausgeschlossen

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@flothi wieso resolved?

Copy link
Collaborator

@flothi flothi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Owner

@Schrolli91 Schrolli91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der Changelog Eintrag fehlt auch noch

@@ -186,7 +186,7 @@ fhemCmd = 0

# for developing - template-module
template = 0

2calendar = 1
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@flothi wieso resolved?

Erstellt eine Icalendar (ICS)-Datei, diese kann in Kalender wie Outlook etc. importiert werden.
Aktuell wird jeder Alarm der via Regex dieses Plugin anspricht, in eine "Gesamtdatei" geschrieben. Denkbar wäre aber auch
für jeden Filter eine seperate Datei anzulegen.
Auch wird aktuell nur ZVEI, aus mangel an Erfahrung, ausgewertet.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@flothi wieso resolved?

@flothi
Copy link
Collaborator

flothi commented Jan 26, 2021

@Schrolli91 Sorry, github zeigte mir hier alles mögliche an - da bin ich wohl falsch abgebogen ;)

@Schrolli91
Copy link
Owner

Nach wie vor würde für einen Merge noch der Changelog Eintrag fehlen

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

Successfully merging this pull request may close these issues.

3 participants