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

Implement IRC client presence (with caveats) #41

Open
hifi opened this issue May 8, 2021 · 4 comments · May be fixed by #254
Open

Implement IRC client presence (with caveats) #41

hifi opened this issue May 8, 2021 · 4 comments · May be fixed by #254

Comments

@hifi
Copy link
Owner

hifi commented May 8, 2021

It would look a lot better if everyone who are on a channel show up "online" rather than "unavailable". We can update the presence status to "offline" if we see a client quit but it's really a best effort as it's only visible to you if that happens on a channel you're in.

The very least we should set all users online that we know are currently on a channel. This can be flushed through a slow background queue.

@piegamesde
Copy link
Contributor

I don't fully understand the description. Is it about bridging a user's presence to IRC or about other users' presence to Matrix? (Does IRC even have some presence-like features?)

@hifi
Copy link
Owner Author

hifi commented May 8, 2021

Showing an IRC puppet online if we have a good guess they are like being on a channel with a Matrix user.

Them showing up as unavailable looks dull on Element so either force every puppet online or do best effort tracking from known events.

If an IRC user is not on any known channel and PMs someone we can set them online and if a PM to IRC user replies with "no such nick" we can set them offline. It isn't perfect but we could also do background checking with WHO if really wanted.

@piegamesde
Copy link
Contributor

I'd say that explicit calls to check if users are online are a bit intrusive towards the server (imagine doing this for all people in a larger room). But a passive "the bridge has seen that persons in the last five minutes" would be good enough. Heck, even simply saying that all people in a room are online (if they weren't they'd left that room) would be fine for me.

@hifi
Copy link
Owner Author

hifi commented May 9, 2021

Of course it doesn't need to do that for people that are in known rooms. It would only need to poll anyone in a DM without a common channel.

Though this is not a high priority feature, mostly just put it up that it should be implemented at some point.

@maxaudron maxaudron linked a pull request Mar 18, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants