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

Push notifications stopped working #1481

Closed
kongo09 opened this issue Oct 3, 2023 · 11 comments · Fixed by #1579
Closed

Push notifications stopped working #1481

kongo09 opened this issue Oct 3, 2023 · 11 comments · Fixed by #1579
Assignees
Labels
A-Push O-Uncommon Most users are unlikely to come across this or unexpected workflow 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

Comments

@kongo09
Copy link

kongo09 commented Oct 3, 2023

Steps to reproduce

  1. Login

Outcome

What did you expect?

Push notifications to arrive on the phone

What happened instead?

No push notifications arriving

Your phone model

Pixel 5

Operating system version

Android 13

Application version and app store

0.2.4-nightly

Homeserver

element.io

Will you send logs?

Yes

Are you willing to provide a PR?

No

@kongo09 kongo09 added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Oct 3, 2023
@kongo09
Copy link
Author

kongo09 commented Oct 4, 2023

Screenshot_20231004-175224

And yes, notifications are enabled for the app

@bmarty bmarty added S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Uncommon Most users are unlikely to come across this or unexpected workflow O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-Push and removed O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Oct 6, 2023
@kongo09
Copy link
Author

kongo09 commented Oct 14, 2023

Just upgraded to Android 14 and the latest nightly. Push still doesn't work and EX is unusable

@bmarty bmarty self-assigned this Oct 16, 2023
@bmarty
Copy link
Member

bmarty commented Oct 16, 2023

I see this log in the rageshake that may cause the issue:

[Push/PushersManager] Unnecessary to register again the same pusher

For some reason the pusher may have been deleted on the server, and in this case, push will not work anymore.

If you have an Element Web instance on the same account, you can check in the settings, you should see a notification target for Element X Android, like below.

image

It's also possible to check this using Element Android: Settings/Advanced settings/Notification targets.

Note: your deviceId is: JZL9Q512qG

Let me know if you have a registered pusher for this device.

@kongo09
Copy link
Author

kongo09 commented Oct 16, 2023

On Desktop:
image

On Android, there is Element given as App Display Name. Nothing about Element X

If this happens, shouldn't this recover itself?

@bmarty
Copy link
Member

bmarty commented Oct 16, 2023

OK, thanks. So I will align the behavior on the iOS one: EXA will register the pusher at each application start.

The only way to recover is to logout and logging again.

@kongo09
Copy link
Author

kongo09 commented Oct 16, 2023

Thanks. I'm happy to try if there is no further investigation needed to figure out why it got lost in the first place. It would be great if the app could test for it's healthy existence without restart and then repair.

@bmarty
Copy link
Member

bmarty commented Oct 16, 2023

why it got lost in the first place

Maybe dig into element.io logs?

if the app could test for it's healthy existence without restart and then repair

This is covered by the notification troubleshoot screen in EA, which does not exist in EA yet, but is listed in #185

@kongo09
Copy link
Author

kongo09 commented Oct 16, 2023

I've now logged out and logged in again. Element Desktop shows:
image

(to be fair, these names don't say much - but at least it increased by another entry...)

Pushes are now coming in again.

@kongo09 kongo09 closed this as completed Oct 16, 2023
@kolaente
Copy link

So once #1579 is merged and released this should work again?

In my Element notification settings I only see Element X on ios (which I also use on my ipad) but none for android.

@bmarty
Copy link
Member

bmarty commented Oct 16, 2023

So once #1579 is merged and released this should work again?

Yes

@p-fruck
Copy link

p-fruck commented Oct 19, 2023

I'm facing the same issue with degoogled Android 13. However, I cannot find any notification targets in the web app or in element x. Are notifications currently only possible when using google play services or should the upcoming release fix this issue too?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Push O-Uncommon Most users are unlikely to come across this or unexpected workflow 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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants