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

feat: add ics feed #67

Merged
merged 7 commits into from
Nov 4, 2023
Merged

feat: add ics feed #67

merged 7 commits into from
Nov 4, 2023

Conversation

24367dfa
Copy link
Member

@24367dfa 24367dfa commented Nov 4, 2023

Derzeit liefert das unter www.netz39.de/events.ics den Feed mit den 3 neusten Events aus.

Offene Punkte:

  • Unter welcher URL wollen wir das ausliefern? - www.netz39.de/feed/eo-events/events.ics
  • Wollen wir noch mehr der Daten aus Variablen ziehen? (Kontakt Mail, Adresse, Start- und Endzeit für die Events)
  • Link auf der Kalenderseite ("abbonier mich als ics feed")
  • Weiterleitung von der "alten" feed URL
  • erzeugten Feed mit https://icalendar.org/validator.html validieren

@24367dfa 24367dfa self-assigned this Nov 4, 2023
@24367dfa 24367dfa linked an issue Nov 4, 2023 that may be closed by this pull request
@24367dfa 24367dfa marked this pull request as draft November 4, 2023 07:41
@24367dfa
Copy link
Member Author

24367dfa commented Nov 4, 2023

nvm... das sind die 3 ältesten events... einfach aus der alphabetischen Liste. Da brauchen wir noch eine andere Lösung.

Ich hab jetzt eine ganze Weile mit jekyll und den Filter/Date compare Funktionen rumprobiert und bin noch nicht glücklich geworden.

tht is a workaround and needs to be changed to a sensibly filtered list
Copy link
Member

@MG-5 MG-5 left a comment

Choose a reason for hiding this comment

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

Je weniger Javascript wir haben, desto besser :D Danke für die Umsetzung, sieht soweit gut aus, nur paar Kleinigkeiten, bezogen auf die Uhrzeit.
Ich wollte das Front Matter des jeweiligen Events noch so ausbauen, dass man hinschreiben kann/muss, wann der Spaß losgeht und wie lange das dauert.

events.ics Outdated Show resolved Hide resolved
events.ics Outdated Show resolved Hide resolved
@MG-5
Copy link
Member

MG-5 commented Nov 4, 2023

nvm... das sind die 3 ältesten events... einfach aus der alphabetischen Liste. Da brauchen wir noch eine andere Lösung.

Ich hab jetzt eine ganze Weile mit jekyll und den Filter/Date compare Funktionen rumprobiert und bin noch nicht glücklich geworden.

Das Problem, was ich bei "3 neuste Eventes anzeigen" sehe ist, dass events.ics nur einmal generiert wird. Erst zum nächsten Commit auf dem Main-Branch haben wir eine aktualisierte .ics

Ich schlage vor, wir beziehen bei jedem Generieren nur die Events ab Buildzeitpunkt ein. Dann sind zumindest die alten Events nicht mehr drin, aber alle zukünftigen.

Chat-GPT sagt da:

---
layout: null
---
BEGIN:VCALENDAR
VERSION:2.0
PRODID:{{ site.url }}
METHOD:PUBLISH{% for event in site.events %}
{% assign eventDate = event.event_date | date: "%Y%m%d" %}
{% assign currentDate = 'now' | date: "%Y%m%d" %}
{% if eventDate >= currentDate %}
BEGIN:VEVENT
.
.
.
``

@penguineer
Copy link
Member

Warum haben wir die Uhrzeiten nicht?

"Z" sehe ich immer noch sehr häufig, aber ich akzeptiere bei Reviews eigentlich keine Zeitstempel ohne Zeitzone mehr, weil das immer mehr zu Problemen führt. (Insbesondere weil beim Hosting die Server inzwischen auch in einer anderen Zeitzone stehen.)

Selbst wenn wir CET hartkodieren müssen, würde ich das trotzdem so reinschreiben und nicht "Z" angeben.

Das könnte auch der Datumsarithmetik helfen.

pages/events_feed_redirect.md Outdated Show resolved Hide resolved
@24367dfa
Copy link
Member Author

24367dfa commented Nov 4, 2023

Warum haben wir die Uhrzeiten nicht?

"Z" sehe ich immer noch sehr häufig, aber ich akzeptiere bei Reviews eigentlich keine Zeitstempel ohne Zeitzone mehr, weil das immer mehr zu Problemen führt. (Insbesondere weil beim Hosting die Server inzwischen auch in einer anderen Zeitzone stehen.)

Selbst wenn wir CET hartkodieren müssen, würde ich das trotzdem so reinschreiben und nicht "Z" angeben.

Das könnte auch der Datumsarithmetik helfen.

weil wir bisher in die events keine Uhrzeiten rein schreiben. Sobald wir das haben, können wir hier auch korrekte Uhrzeiten rein schreiben. Und ja, Z direkt rein zu schreiben ist blöd, das hab ich nur mit kopiert und bisher nicht angepasst.

@24367dfa
Copy link
Member Author

24367dfa commented Nov 4, 2023

@MG-5 hast du eine Idee/einen Vorschlag, wie wir den alten Link auf den Feed am besten auf der events Seite unterbringen?

Das war der Text: (iCal-Feed: https://www.netz39.de/feed/eo-events/)

Ich habe das lokal in meinem Thunderbird abbonieren können, indem ich die Feed-URL vollständig angegeben habe: http://localhost:4000/feed/eo-events/events.ics

@penguineer
Copy link
Member

Warum haben wir die Uhrzeiten nicht?
"Z" sehe ich immer noch sehr häufig, aber ich akzeptiere bei Reviews eigentlich keine Zeitstempel ohne Zeitzone mehr, weil das immer mehr zu Problemen führt. (Insbesondere weil beim Hosting die Server inzwischen auch in einer anderen Zeitzone stehen.)
Selbst wenn wir CET hartkodieren müssen, würde ich das trotzdem so reinschreiben und nicht "Z" angeben.
Das könnte auch der Datumsarithmetik helfen.

weil wir bisher in die events keine Uhrzeiten rein schreiben.

Vllt können wir die dann wirklich dahingehend anpassen, dass sie als All-Day-Events markiert sind und noch die Zeitzone ergänzen?

Sobald wir das haben, können wir hier auch korrekte Uhrzeiten rein schreiben.

Ja, sollten wir immer machen.

Ich weiß auch nicht genau, in welcher Zeitzone die Runner laufen.

Copy link
Member

@penguineer penguineer left a comment

Choose a reason for hiding this comment

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

Ich approve erstmal, damit wir mit dem Thema weiter kommen, aber die Lösung ist noch nicht ideal.

@24367dfa 24367dfa marked this pull request as ready for review November 4, 2023 22:48
@24367dfa
Copy link
Member Author

24367dfa commented Nov 4, 2023

die offenen Punke erledigen wir in separaten PRs

@24367dfa 24367dfa merged commit 36e32ab into main Nov 4, 2023
1 check passed
@24367dfa 24367dfa deleted the feat/ics-feed branch November 4, 2023 22:48
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.

Weiterleitung vom alten Kalenderfeed
3 participants