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 399c2a9 fixes this issue
Commit 7324df1 fixes this issue
Log in to comment on this ticket.