Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

Add Matrix links to community meetings #632

Merged
merged 2 commits into from
Feb 1, 2023
Merged

Conversation

Qalthos
Copy link
Contributor

@Qalthos Qalthos commented Oct 13, 2021

yaml2ical requires the irc key, which fills in the location field on the ics file. This will probably need changes to yaml2ical to get proper matrix links.

Alternately, we can abandon the location field to IRC and put the link to matrix in the description

@sivel
Copy link
Member

sivel commented Oct 13, 2021

Locations should probably be an or, and not only indicating matrix.

@Qalthos Qalthos requested a review from gundalow October 13, 2021 18:31
meetings/ical/testing.ics Outdated Show resolved Hide resolved
meetings/ical/molecule.ics Outdated Show resolved Hide resolved
meetings/lockdown.yaml Outdated Show resolved Hide resolved
meetings/docs.yaml Show resolved Hide resolved
@@ -12,5 +12,5 @@ description: |
schedule:
Copy link
Contributor

Choose a reason for hiding this comment

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

Might need to delete Azure meeting, I'm waiting for confirmation

@samccann
Copy link
Contributor

@Qalthos - DaWGs will be changing our meeting time starting next week. Do you anticipate merging this PR before then (and I can suggest edits on this PR for the new time) or should I open a separate PR and you can rebase after that's merged?

yaml2ical requires the `irc` key, which fills in the location field on
the ics file.
@Qalthos Qalthos changed the title First go at moving meetings to Matrix Add Matrix links to community meetings Jan 30, 2023
Copy link
Contributor

@oraNod oraNod left a comment

Choose a reason for hiding this comment

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

DaWGs details LGTM

@Andersson007 Andersson007 merged commit a7abf76 into ansible:main Feb 1, 2023
@Andersson007
Copy link
Contributor

thanks everyone!

@Qalthos Qalthos deleted the matrix branch February 1, 2023 14:51
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.

6 participants