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

Notifications are broken on 1.3.14 F-Droid #5003

Closed
konomikitten opened this issue Jan 20, 2022 · 12 comments · Fixed by #5012
Closed

Notifications are broken on 1.3.14 F-Droid #5003

konomikitten opened this issue Jan 20, 2022 · 12 comments · Fixed by #5012
Assignees
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 Something isn't working: bugs, crashes, hangs and other reported problems Z-fdroid F-Droid variant

Comments

@konomikitten
Copy link

konomikitten commented Jan 20, 2022

Steps to reproduce

  1. Enable Caffeine for testing purposes this prevents your phone sleeping.
  2. Observe that no notifications come through when Element is closed and not running as a background app, even though the "Listening for notifications" persistent notification is present.
  3. Launch Element and then press the home button to move it to the background (running as a background app), note that no notifications come through.
  4. Finally bring Element to the foreground but do not have it focused on a chat that would generate a notification and you will finally have notifications displayed.

Summing up, currently for the F-Droid build notifications are broken in every aspect unless Element is in the foreground.

The problem was not occurring on the previous Element build from F-Droid (1.3.9).

Outcome

What did you expect?

Notifications should be received no matter what state Element is in.

What happened instead?

No notifications are received unless the app is open and in the foreground.

Your phone model

Asus Zenfone Selfie ZD551KL

Operating system version

Android 7.1.2 (LineageOS 14.1)

Application version and app store

Element version 1.3.14 olm version 3.2.10 from F-Droid

Homeserver

matrix.org

Your phone model

Motorola Moto E Dual SIM (2nd gen) XT1521

Operating system version

Android 10 (LineageOS 17.1)

Application version and app store

Element version 1.3.14 olm version 3.2.10 from F-Droid

Homeserver

matrix.org

Will you send logs?

Yes

@konomikitten konomikitten added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Jan 20, 2022
@konomikitten
Copy link
Author

Both phones have sent rage shake reports with the heading and issue number of this bug report.

@fabienli
Copy link

same issue on my samsung galaxy s9

@ouchadam
Copy link
Contributor

does this also occur when Caffeine is not running?

@konomikitten
Copy link
Author

@ouchadam yes still a problem without caffeine on.

@ouchadam ouchadam added A-Notifications Z-fdroid F-Droid variant 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 labels Jan 20, 2022
@ouchadam ouchadam self-assigned this Jan 20, 2022
@ouchadam
Copy link
Contributor

thanks for raising, it looks like there's been a regression which caused the optimised for battery mode to no longer schedule the next sync after completing the first one in the background, a temporary workaround is to select the optimised for realtime in the Settings -> Notifications -> Background sync mode

@konomikitten
Copy link
Author

a temporary workaround is to select the optimised for realtime in the Settings -> Notifications -> Background sync mode

How heavily will this drain a phones battery compared to optimised for battery?

@ouchadam
Copy link
Contributor

it's hard to say as different ROMs/OEMs have unique battery optimisations, for most cases where the device is being used throughout the day there isn't too much of a difference, if a device is sat idle and not being used at all then there would be a much bigger difference in battery usage.

@konomikitten
Copy link
Author

@ouchadam pretty sure once I changed the sync setting and leave Element open on a channel like say #matrix that it just started consuming tons of cpu and started leaking memory please advise. Screenshot of Element resource hogging on my phone via adb shell htop:

Screenshot_2022-01-21_11-46-16

@bmarty
Copy link
Member

bmarty commented Jan 21, 2022

This issue will be fixed in 1.3.16.
I open a dedicated issue for your latest concern @konomikitten

@Karamelmar
Copy link

Got the notifications running again after switching "Background Sync" in Settings > Notifications from "Optimized for battery" to "Optimized for real time".

@marcusmueller
Copy link

@Karamelmar this did not fix the issue for me; 1.3.15 android via fdroid.

Neither did giving unrestricted network access. This is a rather severe regression.

@damnms
Copy link

damnms commented Feb 2, 2023

same here, just noticed a message sent 3 days ago from my mom. she called me on my phone asking why i dont respond to the message... nice

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 Something isn't working: bugs, crashes, hangs and other reported problems Z-fdroid F-Droid variant
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants