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

Presence status is completely wrong #5727

Closed
turt2live opened this issue Nov 29, 2017 · 4 comments
Closed

Presence status is completely wrong #5727

turt2live opened this issue Nov 29, 2017 · 4 comments
Labels
A-Presence P2 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@turt2live
Copy link
Member

Description

For some rooms I'm offline, some I'm away, and some can't decide while I'm typing. Something is very wrong with the logic.

Note: I'll look into this as part of my 'fix riot' sprint

Version information

  • Platform: web (in-browser)
  • Browser: Chrome 62
  • OS: Windows 10
  • URL: riot.im/develop
@lampholder
Copy link
Member

I'm guessing you mean on a homeserver that is configured with presence enabled?

@lampholder lampholder added T-Defect S-Major Severely degrades major functionality or product features, with no satisfactory workaround P2 labels Dec 1, 2017
@turt2live
Copy link
Member Author

Yes, my own in this case. For reference, here's some of the behaviour I'm seeing:

  • Randomly considered "offline" when switching rooms, never updating to "away" or even "online"
  • Occasionally considered "away" even while actively using the app

It's possible this is the result of riot-web conflicting with another app/client (like another browser or even my android phone). The expected behaviour is:

  • Show as "Offline" only when the user has specifically asked to be shown as offline (or when the app is closed)
  • Show as "Away" when the user hasn't used the app in a while
  • Show as "Online" when the user is currently using the app. Ie: typing, scrolling, moving around, etc.

@turt2live
Copy link
Member Author

This is pretty bad.
image

It's probably best to back out of the PR I wrote for prensece management. I'll open a PR sometime this week(end) to do just that.

@dbkr
Copy link
Member

dbkr commented Apr 12, 2018

This should be fixed now #5881 and matrix-org/matrix-react-sdk#1676 are merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Presence P2 S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

No branches or pull requests

4 participants