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

Missing push notifications #1655

Closed
stefanceriu opened this issue Sep 8, 2023 · 2 comments
Closed

Missing push notifications #1655

stefanceriu opened this issue Sep 8, 2023 · 2 comments
Labels
A-Notifications O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@stefanceriu
Copy link
Member

Original report

I tried around with notifications a bit because I was missing some of them. What I noticed is this: Assume you have the app open and have a room open with alice. You talk but at some point you lock your phone, with the room still open in the background. Now when Alice sends you two messages, one with: "hello" and one with "how are you", I can observe a notification for "hello" but as soon as the "how are you" message comes through, the notification with "hello" gets removed and I am only left with the "how are you" notification.

https://github.com/vector-im/element-x-ios-rageshakes/issues/856

@stefanceriu stefanceriu added S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Occasional Affects or can be seen by some users regularly or most users rarely A-Notifications labels Sep 8, 2023
@stefanceriu
Copy link
Member Author

After talking with @pixlwave about this what believe happens is that the app keeps running in the background (because of the 30sec sync task) and it keeps marking newly received messages as read, as if the user actually saw them.

We should observe the application state and stop sending read receipts for new messages while the app is not in the foreground

@manuroe
Copy link
Member

manuroe commented Nov 15, 2023

Dup of #1923. Notifications that disappeared is part of the symptom we observed when fixing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Notifications O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

No branches or pull requests

3 participants