#213 RFE: support Matrix meeting links
Closed: Fixed by humaton. Opened 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

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?

Thanks a lot @james !
@smilner happy to do so, see https://pagure.io/fedocal/pull-request/214#comment-202707

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)

Commit 399c2a97 fixes this issue

Commit 7324df17 fixes this issue

Commit 399c2a97 fixes this issue

Metadata
Related Pull Requests