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 day/time of weekly meeting to the README #6855

Merged
merged 1 commit into from
May 21, 2018
Merged

Add day/time of weekly meeting to the README #6855

merged 1 commit into from
May 21, 2018

Conversation

chrisvanpatten
Copy link
Contributor

@chrisvanpatten chrisvanpatten commented May 20, 2018

Description

Added the day and time (Wednesday at 13:00 UTC) of the weekly meeting to the README. As far as I can tell the only place this is listed is in the sidebar of make/core, which is a bit hard to find if you don't know it's there.

(I also considered adding a timeanddate.com link for converting the time zone, but the links they generate require including a specific date so it would have to be updated weekly. Not ideal.)

How has this been tested?

n/a

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.

@danielbachhuber danielbachhuber added the [Type] Developer Documentation Documentation for developers label May 20, 2018
@danielbachhuber danielbachhuber added this to the 3.0 milestone May 20, 2018
@gziolo gziolo merged commit e38453f into WordPress:master May 21, 2018
@aduth
Copy link
Member

aduth commented May 24, 2018

Should we just point to the "Meetings - Make WordPress" page instead, to limit the number of places we need to update when the end of daylight savings time rolls around again?

Related: https://make.wordpress.org/core/2018/03/21/core-editor-meeting-time-shift/

Case in point: The Editor component page still shows the following outdated text:

💬 Weekly chat Wednesday, May 30, 2018, 10:00 AM EDT in the #core-editor Slack channel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants