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 notifications badges #25331

Open
Tracked by #24392
neilisfragile opened this issue May 10, 2023 · 9 comments
Open
Tracked by #24392

Missing notifications badges #25331

neilisfragile opened this issue May 10, 2023 · 9 comments
Labels
A-Notif-Panel 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 X-Regression

Comments

@neilisfragile
Copy link
Member

neilisfragile commented May 10, 2023

Steps to reproduce

Relaying bug report from @rxl881

image

https://github.com/matrix-org/element-web-rageshakes/issues/21478

Outcome

What did you expect?

Notification badges to be present where there are unread messages in notifying rooms

What happened instead?

Missing notification badges meaning that the user missed messages.

Operating system

Mac

Application version

No response

How did you install the app?

No response

Homeserver

vector.modular.im

Will you send logs?

https://github.com/matrix-org/element-web-rageshakes/issues/21478

@Johennes
Copy link
Contributor

@rxl881 do you have any recollection of when approximately you did notice this issue occur for the first time?

@andybalaam / @weeman1337 I know this is the opposite problem of what we've been combating in #24392 but could this in any way be related to our recent fixes for stuck unreads? We haven't yet received any similar reports to my knowledge but it would be good if we could do an initial investigation to verify if this is a regression.

@andybalaam
Copy link
Member

Thanks @Johennes we've had a look and we don't think this could have been caused by our recent changes, although we can't dismiss the possibility that we have somehow revealed an existing bug - if so, we don't know how!

I do think we should investigate this as part of the work we are doing on stuck unreads/notifs though.

@germain-gg
Copy link
Contributor

I think I have experienced this too.
I just had two audible notifications in Element, and no increase in unread count anywhere, nor any room was highlighted.

Opening the notification panel didn't help, this presumably happened in an encrypted room.

my investigation has not yielded much results so far

@rxl881
Copy link
Contributor

rxl881 commented May 16, 2023

Hi @Johennes -- I believe that I first saw this on May 2nd, after coming back from the early May bank holiday in the UK. I then saw the same on May 9th, after coming back from the next BH.

@Johennes
Copy link
Contributor

Ok, so fairly recently which means it could possibly relate to the fixes we've made under #24392.

@weeman1337 weeman1337 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 May 30, 2023
@weeman1337 weeman1337 removed their assignment May 30, 2023
@rxl881
Copy link
Contributor

rxl881 commented Jun 5, 2023

I just returned from holiday and got bitten by this again (unread notification badges missing from rooms) :( , using Element Nightly version: 2023041901
Olm version: 3.2.12

@Johennes
Copy link
Contributor

Johennes commented Jun 5, 2023

@rxl881 that version seems quite old actually. I'm on 2023060501. We've had an update bug on nightly a few weeks ago that required people to manually download and install the update once from https://packages.element.io/nightly/update/index.html. Could you please do that and let us know if you still experience the issue?

@rxl881
Copy link
Contributor

rxl881 commented Jun 5, 2023

Thanks. I'll try manually updating with that package now 👍

@robintown
Copy link
Member

#25679 may be related

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Notif-Panel 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 X-Regression
Projects
None yet
Development

No branches or pull requests

8 participants