search

matrix-org / matrix-appservice-irc

Node.js IRC bridge for Matrix
Apache License 2.0
462 stars 151 forks source link

Add IRC users and rooms to a space. #1591

Open ara4n opened 2 years ago

ara4n commented 2 years ago

When we had groups, we added IRC users/rooms to that group in order for flair to work.

Nowadays we have spaces, and it feels like we're massively missing a trick by not letting users automatically group their bridged convos together based on the network they're bridged from. Can we add to spaces by default?

Half-Shot commented 2 years ago

IRC users were never actually added to groups, there was voodoo in Synapse which would mark users inside an appservice as part of a group for performance reasons. I think trying to join our 100,000 or so libera users to a room would cost us an arm and a leg in performance cost for little benefit.

For rooms, we can probably just do it.

Mikaela commented 2 years ago

Doesn't this kind of depend on https://github.com/matrix-org/matrix-spec-proposals/pull/3219 to be useful?

Half-Shot commented 2 years ago

I suspect it still has use in terms of allowing your client to organize your rooms (and, performance issues not withstanding, users) nicely into categories.

morguldir commented 2 years ago

I also think that this could be useful for showing how realistic the space flairs MSC is and how well a implementation works, if it doesn't seem realistic then perhaps this comment is the better option? https://github.com/matrix-org/matrix-spec-proposals/pull/3219#issuecomment-1081838263 (probably should have been a thread)

If you think it's too expensive to be realistic then maybe leave a comment on the MSC