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

Can't receive notifications #1887

Open
Te-Z opened this issue Jun 18, 2024 · 16 comments
Open

Can't receive notifications #1887

Te-Z opened this issue Jun 18, 2024 · 16 comments
Assignees
Labels
blocker bug Something isn't working FT::NOTIFICATION

Comments

@Te-Z
Copy link
Contributor

Te-Z commented Jun 18, 2024

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

@Te-Z Te-Z added FT::NOTIFICATION bug Something isn't working labels Jun 18, 2024
@Te-Z Te-Z self-assigned this Jun 19, 2024
@Te-Z
Copy link
Contributor Author

Te-Z commented Jul 5, 2024

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

@Te-Z
Copy link
Contributor Author

Te-Z commented Jul 5, 2024

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

@Te-Z
Copy link
Contributor Author

Te-Z commented Jul 12, 2024

@Te-Z
Copy link
Contributor Author

Te-Z commented Jul 12, 2024

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
Copy link
Contributor Author

Te-Z commented Jul 13, 2024

sometimes no notifications at all on Chromium

Capture.video.du.13-07-2024.08.07.32.webm

@Te-Z
Copy link
Contributor Author

Te-Z commented Jul 13, 2024

Opera

Capture.video.du.13-07-2024.08.11.59.webm

@nqhhdev
Copy link
Member

nqhhdev commented Jul 15, 2024

What happens on mobile and web?

IMO:

  • We need to focus on mobile first (P1) → Find root cause and solution
  • Note for each browser on web (P2) → Create other tickets

@Te-Z
Copy link
Contributor Author

Te-Z commented Jul 15, 2024

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

@Te-Z
Copy link
Contributor Author

Te-Z commented Jul 17, 2024

@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
Copy link
Contributor Author

Te-Z commented Jul 17, 2024

@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:
#1954 (comment)

@Te-Z
Copy link
Contributor Author

Te-Z commented Jul 25, 2024

Suggestion:

  • how to update counter live on tab (web) -> without enabling notification

@tprudentova
Copy link
Collaborator

video_2024-08-16_11-56-43.mp4

The problem with notifications reappeared on Android this week :(

@guimard
Copy link
Member

guimard commented Aug 21, 2024

video_2024-08-16_11-56-43.mp4

The problem with notifications reappeared on Android this week :(

Confirmed, no more any notification

@hoangdat
Copy link
Member

hoangdat commented Aug 28, 2024

DoD:

  • during vacation: no push notification -> debug the PUSH Gateway server
  • monitoring the number of notificaiton
  • ...

@hoangdat
Copy link
Member

work with BE-DevOps team to handle it with #1640

@hoangdat hoangdat self-assigned this Sep 27, 2024
@hoangdat
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker bug Something isn't working FT::NOTIFICATION
Projects
None yet
Development

No branches or pull requests

5 participants