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

Read Flux meeting ical #452

Merged
merged 7 commits into from
Aug 16, 2021
Merged

Read Flux meeting ical #452

merged 7 commits into from
Aug 16, 2021

Conversation

dholbach
Copy link
Member

@dholbach dholbach commented Aug 10, 2021

Read the cncf-flux-dev calendar (ical) and show the events in the upcoming month.

Fixes: #449

Things that still need doing:

  • link to the calendar so folks know how to find out more (Daniel)
  • do timezone arithmetic so everything is in one TZ
  • make this look pretty (@juozasg)
  • decide as a group that we want to also keep track of talks, etc in our cncf-flux-dev calendar

- [ ] make sure everyone involved has access to the calendar (moved to fluxcd/community#113)
- [ ] hook up calendar include generation with daily cronjob (should be handled by https://github.com/fluxcd/website/blob/main/.github/workflows/netlify.yml)
- [ ] figure out if we can link to the events in question... do we get the data in the ICS somewhere (Daniel) - moved to #461
- [ ] make links clickable in the event description - moved to #461
- [ ] have a mouse-over with event data (@juozasg + Daniel) - moved to #461
- [ ] maybe convert to browser TZ(?) - moved to #461

@dholbach dholbach changed the title Add infrastructure to read from Flux ical Read Flux meeting ical Aug 10, 2021
@dholbach
Copy link
Member Author

See the calendar table in the community section of https://deploy-preview-452--fluxcd.netlify.app/

We could do a mouse-over which shows you the event description as well, cf https://lists.cncf.io/g/cncf-flux-dev/calendar

@dholbach
Copy link
Member Author

dholbach commented Aug 11, 2021

@dholbach dholbach marked this pull request as ready for review August 13, 2021 11:56
@dholbach dholbach mentioned this pull request Aug 13, 2021
4 tasks
@dholbach dholbach requested a review from alisondy August 13, 2021 11:58
@dholbach
Copy link
Member Author

Part 2 is #461

Daniel Holbach and others added 7 commits August 16, 2021 10:17
	Fixes: #449

Signed-off-by: Daniel Holbach <daniel@weave.works>
Signed-off-by: Daniel Holbach <daniel@weave.works>
Signed-off-by: Daniel Holbach <daniel@weave.works>
Signed-off-by: Daniel Holbach <daniel@weave.works>
Signed-off-by: Daniel Holbach <daniel@weave.works>
Signed-off-by: Juozas <juozasgaigalas@gmail.com>
Signed-off-by: Daniel Holbach <daniel@weave.works>
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @dholbach

@dholbach dholbach merged commit 5d92509 into fluxcd:main Aug 16, 2021
@dholbach dholbach deleted the fix-449 branch August 16, 2021 14:00
@scottrigby
Copy link
Member

A minute too late, these look great! @dholbach 👏

juozasg pushed a commit to juozasg/website that referenced this pull request Aug 30, 2021
	Please merge after fluxcd#452

Signed-off-by: Daniel Holbach <daniel@weave.works>
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.

Add events to landing page
4 participants