#213 RFE: support Matrix meeting links
Closed: Fixed 6 months ago by humaton. Opened a year ago by salimma.

A lot of meetings are migrating from IRC to Matrix, please support Matrix meeting room links (either #room:host.name.tld or just generic https URLs so we can put in https://matrix.to/#/#room:host.name.tld)

Bonus if we can specify the #room:host.name.tld format and this is automatically turned into a HTML link with that displayed and the matrix.to link as the href


Mailing list discussion about it: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/SYOIFHJXSX7YKEV25K2EOWQKJCXIGWMR/

Back in the day a filter to disallow # in meeting locations was added (https://pagure.io/fedocal/issue/118), that's the starting point to make it matrix compatible.

Couldn't invest much time right now but I think changing the code to use a RegEx would be the best. Valid channel/room links in the meeting location:
- ircroom@irc.server
- #room:host.name.tld
- https://matrix.to/#/#room:host.name.tld

Or drop the validation completely as hacky workaround...
- https://pagure.io/fork/wombelix/fedocal/blob/master/f/fedocal/api.py#_222
- https://pagure.io/fork/wombelix/fedocal/blob/master/f/fedocal/forms.py#_58

Metadata Update from @zlopez:
- Issue tagged with: easyfix

9 months ago

I can take a look at this ... don't seem to be able to assign myself atm.

I have a change which validates against just the three formats above:

https://pagure.io/fedocal/pull-request/214

Awesome, thanks @james!

@wombelix Mind doing a review of the changes and providing feedback?

Fedora DEI team use the field with https://meet.google.com/, so this should be added to the regexp.

See https://calendar.fedoraproject.org/diversity-team/#m10661

(ok, didn't see the others patchs that relaxed the regexp)

Log in to comment on this ticket.

Metadata
Related Pull Requests
  • #222 Merged 5 months ago
  • #214 Merged 6 months ago