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

Add a .ics to the site #439

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

Add a .ics to the site #439

wants to merge 4 commits into from

Conversation

jasondlee
Copy link

Add .ics template
Create data file for dates

_data/schedule.yaml Outdated Show resolved Hide resolved
calendar.ics Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Because there are a few entries I'll comment here. Is the timezone stuff necessary? If so we should probably use GMT as the base.

Copy link
Author

Choose a reason for hiding this comment

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

Good question. When we say "code freeze on mm/dd", which TZ do we use to mark the end of 'dd'?

Copy link
Contributor

Choose a reason for hiding this comment

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

US Central time. :)

I'm only 1/2 kidding. We don't use Central time as we're in flyover country and are not worthy of such prominence, but the reality is the deadlines are roughly end of business day my time. Unless it's about a micro James is coordinating, in which case it's EOBD his time. :)

I don't think putting specific times is good, although I think the way you've allowed for it with an if block is good. If TZ data can somehow be read as implying a time even if one isn't specified, UTC works because end of calendar day UTC is 6 hours before end of business day Central time, which is close enough.

Copy link
Author

@jasondlee jasondlee Aug 8, 2023

Choose a reason for hiding this comment

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

I've made an attempt to update the TZ to GMT. It looks a little odd, but that's what The Internet seems to think I should use. I did validate it using ical4j's CLI tools, and it's happy. I can also import the .ics from a test server locally into Thunderbird, and the calendar renders as expected. Or seems to. :)

_data/schedule.yaml Outdated Show resolved Hide resolved
_data/schedule.yaml Outdated Show resolved Hide resolved
@bstansberry
Copy link
Contributor

@jasondlee Is there any public discussion around this anywhere? It seems like a good idea, but I also know various people are thinking about doing things in this broader community communication area, so I don't want to do something that messes with other plans.

If you're proposing this as part of a group effort can you @ ping the relevant people here and request a review?

@jasondlee
Copy link
Author

@bstansberry Not really, no. I think I had intended this to start that discussion (or join something already going?) I'll email the PR link to the dev list to give it more visibility in case there's a conversation I'm not privy to.

Remove test data
Set TZ to GMT
Add ORGANIZER required field (per ical4j validation)
@darranl
Copy link
Contributor

darranl commented Aug 23, 2023

I like having a published calendar but could we somehow automate generate publication from a SmartSheet?

I have used an internal SmartSheet to generate a set of dates, not because the data needs to be internal to Red Hat but just because as a tool it calculates dependencies and relative dates etc - if we could export that it would mean we would not need manual synchronisation.

Copy link
Contributor

@darranl darranl left a comment

Choose a reason for hiding this comment

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

I think we should continue to discuss how this can be generated maintained so we can keep it up to date? Maybe start a discussion in chat to follow up?

@jasondlee
Copy link
Author

I think we should continue to discuss how this can be generated maintained so we can keep it up to date? Maybe start a discussion in chat to follow up?

Agreed. Everybody got busy with Other Stuff and this got dropped. I'd like to see it finished. I'll follow up in chat.

Copy link
Member

@rhusar rhusar left a comment

Choose a reason for hiding this comment

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

I think we should continue to discuss how this can be generated maintained so we can keep it up to date? Maybe start a discussion in chat to follow up?

Agreed. Everybody got busy with Other Stuff and this got dropped. I'd like to see it finished. I'll follow up in chat.

Did the chat end up being created? I always assumed we are just going to create a public Google calendar, embed / link it from the website. Primarily for the simplicity of management by team members and all the existing functionality to add to calendar, notifications, etc.

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.

None yet

5 participants