linagora / twake-on-matrix

A multi-platform (Flutter) application for connecting to Matrix - an open network for secure, decentralized communication, with your favorite devices.
GNU Affero General Public License v3.0
46 stars 14 forks source link

Can't receive notifications #1887

Open Te-Z opened 1 month ago

Te-Z commented 1 month ago

People complain from not being able to receive notifications. We need to investigate this

Te-Z commented 4 weeks ago

How it works right now: sender -> sygnal -> firebase -> receiver

Te-Z commented 4 weeks ago

On iOS work on the version on testflight but do not work on the version from the store.

Te-Z commented 3 weeks ago

image

https://www.notion.so/linagora/What-we-can-reproduce-write-now-a3822a83b6854949b2aa324890ac1b7a?pvs=4

Te-Z commented 3 weeks ago

Hypothesis: problems with https://pub.dev/packages/flutter_local_notifications (v. 17.1.2 used in twake chat 2.5.9) -> used only for mobile

On web notifications use JS

Te-Z commented 3 weeks ago

sometimes no notifications at all on Chromium

Capture vidéo du 13-07-2024 08:07:32.webm

Te-Z commented 3 weeks ago

Opera

Capture vidéo du 13-07-2024 08:11:59.webm

nqhhdev commented 2 weeks ago

What happens on mobile and web?

IMO:

Te-Z commented 2 weeks ago

@nqhhdev no problem on mobile with v2.5.9 (see table above)

Te-Z commented 2 weeks ago

@Bobpodvalnyi I think we should display something explicit like Whatsapp when the notification permission is not granted. On some browsers the notification permission native dialog is not displayed on click. This would avoid some complaints. WDYT ?

notification permission

image

Te-Z commented 2 weeks ago

@Bobpodvalnyi you can see here how the app behaves on different browsers. On each browser it's really different, that's why I think we need an explicit indication: https://github.com/linagora/twake-on-matrix/pull/1954#issue-2412940079

Te-Z commented 1 week ago

Suggestion: