Skip to content
This repository has been archived by the owner on Feb 3, 2025. It is now read-only.

Add calendar/event list #30

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add calendar/event list #30

wants to merge 3 commits into from

Conversation

sbruder
Copy link
Collaborator

@sbruder sbruder commented Jul 23, 2022

It is implemented by making it possible to assign event metadata to an announcement like this:

---
title: Test announcement
date: 2022-07-18
description: "This will be shown in the iCal event"
extra:
  event_start: 2022-07-19T10:00:00Z
  event_end: 2022-07-19T11:00:00Z
---
Description of the announcement/event only shown on the website

If it is not a good way to have overlap with the announcement (which has the advantage that they are also visible on the main page), we could also separate it.

The calendar overview page is very simple, because implementing a calendar display from scratch is not very easy.

Since there currently are no calendar events, I added some fake ones to this test deployment: https://dapper-swan-e898e3.netlify.app/

@sbruder sbruder requested a review from MMesch July 23, 2022 10:15
@github-actions
Copy link

{% for page in section.pages -%}
{% if page.extra.event_start and page.extra.event_end -%}
BEGIN:VEVENT
SUMMARY:{{ page.title }}
Copy link

Choose a reason for hiding this comment

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

Is this template going to emit HTML entities?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks, I didn’t notice that. I changed it to not escape it anymore, so it should now include it raw.

@github-actions
Copy link

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

Successfully merging this pull request may close these issues.

2 participants