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

telegram #2298

Closed
dimka40185 opened this issue Apr 6, 2024 · 8 comments · Fixed by #2344
Closed

telegram #2298

dimka40185 opened this issue Apr 6, 2024 · 8 comments · Fixed by #2344
Labels

Comments

@dimka40185
Copy link

telegram with micro g does not work correctly, notifications come but without sound and banner, only the number on the telegram icon. The microg is configured correctly
Screenshot_20240406_124339_com google android gms
Screenshot_20240406_124335_com google android gms
Screenshot_20240406_124328_com google android gms
Screenshot_20240406_124323_com android settings

@dimka40185 dimka40185 added the bug label Apr 6, 2024
@chpio
Copy link

chpio commented Apr 8, 2024

How is this related to microg? Seems to me like telegram is not showing the notifications properly. Have you checked the telegram notification settings?

@dimka40185
Copy link
Author

How is this related to microg? Seems to me like telegram is not showing the notifications properly. Have you checked the telegram notification settings?

This is due to the Microg, as it does not wake up the telegram when a push notification is received.

@Sapiosenses
Copy link

How is this related to microg? Seems to me like telegram is not showing the notifications properly. Have you checked the telegram notification settings?

This is due to the Microg, as it does not wake up the telegram when a push notification is received.

I have been using Telegram with microG for years, it sends notifications fine.

Caveat: for a variety of reasons including better privacy, I use the "Telegram FOSS" fork that is maintained by the F-droid team instead of the official client. Telegram FOSS does not rely on FCM ("Cloud Messaging") to send notifications.

That said, I also know various people that use the official client and it sends notifications fine for them as well.

Lastly, you cut off the part of the screenshot that indicates if you have battery optimizations ignored for microG. If that is not set properly, modern Android ROMs will keep trying to kill the background microG processes, which will impact things like push notifications.

@matteobtt
Copy link
Contributor

Is due to the fact that Telegram does not show notifications if they are already dismissed

https://github.com/DrKLO/Telegram/blob/a906f12aaec2768969c77650a7e4b377baa6cf2a/TMessagesProj/src/main/java/org/telegram/messenger/NotificationsController.java#L3776-L3779

The problem is that microG does not forward google.sent_time to Telegram and so this variable is set to 0 causing the if to be always executed.

@gcvl
Copy link

gcvl commented Apr 27, 2024

Is due to the fact that Telegram does not show notifications if they are already dismissed

https://github.com/DrKLO/Telegram/blob/a906f12aaec2768969c77650a7e4b377baa6cf2a/TMessagesProj/src/main/java/org/telegram/messenger/NotificationsController.java#L3776-L3779

The problem is that microG does not forward google.sent_time to Telegram and so this variable is set to 0 causing the if to be always executed.

This has never been investigated in nearly a decade, see #1986 .
Can you place a pull request and attempt a fix?

@Vavun
Copy link
Contributor

Vavun commented Apr 27, 2024

@DaVinci9196 help please ?

The problem is that microG does not forward google.sent_time to Telegram and so this variable is set to 0 causing the if to be always executed.

@matteobtt
Copy link
Contributor

This has never been investigated in nearly a decade, see #1986 . Can you place a pull request and attempt a fix?

@gcvl Yes, #2344

@gcvl
Copy link

gcvl commented Apr 28, 2024

This has never been investigated in nearly a decade, see #1986 . Can you place a pull request and attempt a fix?

@gcvl Yes, #2344

Wow, grande Matteo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants