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

doc(README): add notes on timestamps for events #26

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

laurelmay
Copy link

@laurelmay laurelmay commented Jan 25, 2024

The January meetup was shown as being at 4:30PM ET due to the UTC offset in the front matter being -04:00 while the current east coast time is -05:00. This corrects that for this event (even though it's over) and adds a note to the README and the event-frontmatter.txt template file about the correct offsets for Eastern Time, which is likely to be the most relevant timezone for the community.

The formatting is done client side as well but I am not sure whether there is conversion to the user's
local timezone.

@CLAassistant
Copy link

CLAassistant commented Jan 25, 2024

CLA assistant check
All committers have signed the CLA.

The January meetup was shown as being at 4:30PM ET due to the UTC offset
in the front matter being `-04:00` while the current east coast time is
`-05:00`. This corrects that for this event (even though it's over) and
adds a note to the README and the `event-frontmatter.txt` template file
about the correct offsets for Eastern Time, which is likely to be the
most relevant timezone for the community.

The conversion to local timezone is done in JavaScript so it doesn't
really matter what the offset is so long as the overall time is correct
but thinking in east coast time is probably the mostly likely mindset
for contributors (and causes less friction than requiring all times be
specified in UTC).
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.

2 participants