Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add IRC users and rooms to a space. #1591

Open
ara4n opened this issue Aug 11, 2022 · 4 comments
Open

Add IRC users and rooms to a space. #1591

ara4n opened this issue Aug 11, 2022 · 4 comments
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements.

Comments

@ara4n
Copy link
Member

ara4n commented Aug 11, 2022

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
Copy link
Contributor

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.

@Half-Shot Half-Shot added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements. label Aug 11, 2022
@Mikaela
Copy link
Contributor

Mikaela commented Aug 11, 2022

Doesn't this kind of depend on matrix-org/matrix-spec-proposals#3219 to be useful?

@Half-Shot
Copy link
Contributor

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
Copy link

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? matrix-org/matrix-spec-proposals#3219 (comment) (probably should have been a thread)

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements.
Projects
None yet
Development

No branches or pull requests

4 participants