element-hq / element-hq/element-android

Home Notifications badge overinflated if room in multiple spaces

Open
#4,119 2 comments 0 reactions 1 assignee Claimed by @fedrunov View on GitHub
A-Notifications A-Spaces O-Occasional S-Major T-Defect
Dominant language
Kotlin
Stars
3.7k
Forks
917
PR merge metrics
No merged PRs in 30d

Description

### Steps to reproduce

1. Receive notifications in room which is in multiple spaces
2. Switch to Home space

### What happened?

### What did you expect?

Expected the count to be accurate

### What happened?

The count is multiplied by the number of spaces the room is in

![image](https://user-images.githubusercontent.com/2403652/135294196-79a48b49-b67b-4b2c-9202-1ff8274a26ad.png)
![image](https://user-images.githubusercontent.com/2403652/135294212-aad86cff-882d-4e30-88a7-a2c980782531.png)

See count top left is 2, but the notification in both of the Spaces is actually in the same room.

### Your phone model

Samsung S20U

### Operating system version

_No response_

### Application version and app store

_No response_

### Homeserver

_No response_

### Have you submitted a rageshake?

No

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.