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

[Android] Notification Dot not showing up #2619

Closed
ajdev6 opened this issue Mar 4, 2019 · 8 comments
Closed

[Android] Notification Dot not showing up #2619

ajdev6 opened this issue Mar 4, 2019 · 8 comments

Comments

@ajdev6
Copy link

ajdev6 commented Mar 4, 2019

Summary

Notification Dot is not showing up on app icon.

Environment Information

  • Device Name: OnePlus 6
  • OS Version: 9
  • Mattermost App Version: 1.16.1
  • Mattermost Server Version: 5.9.0

Steps to reproduce

How can we reproduce the issue?
When a user sends you a message and app is running in background, notification dot is not showing on app icon.

Show Notification Dot is enabled in app settings

Expected behavior

When a push notification is received and app is running in background , notification dot should appear on app icon.

Observed behavior

No notification dot shows up.
whatsapp image 2019-03-04 at 18 47 17

Possible fixes

If you can, link to the line of code that might be responsible for the problem.

@amyblais
Copy link
Member

amyblais commented Mar 4, 2019

Hi @ajdev6, would you happen to know your Mattermost server version? You would normally find the information from Mattermost settings > About this App section.

@ajdev6
Copy link
Author

ajdev6 commented Mar 5, 2019

Hi @ajdev6, would you happen to know your Mattermost server version? You would normally find the information from Mattermost settings > About this App section.

5.9.0

@amyblais
Copy link
Member

amyblais commented Mar 6, 2019

Hi @ajdev6,

Letting you know that this is not a bug as the badges on Android are only available on certain devices. Android officially added support for 8.0+ https://developer.android.com/training/notify-user/badges.

Would you like to share this idea on our feature request forum here: https://mattermost.uservoice.com/forums/306457-general?

@ajdev6
Copy link
Author

ajdev6 commented Mar 6, 2019

Hi @amyblais,

I am testing on OnePlus 6 on Android 9. It supports notification badge. I have diagnosed the issue.
The issue is with mattermost-server payload.

I checked with the server payload in this file https://github.com/mattermost/mattermost-push-proxy/blob/master/server/push_notification.go

We need to update it according to the following docs:
https://firebase.google.com/docs/cloud-messaging/android/receive#sample-receive

https://firebase.google.com/docs/cloud-messaging/concept-options#notifications_and_data_messages

Its not a feature request. The shortcutBadger is already implemented in react native notifications module.

Hope this helps!

@amyblais
Copy link
Member

amyblais commented Mar 6, 2019

Hi @ajdev6, our developers said that this is a new thing that we have never added support for. Only some devices have had this feature.

@amyblais
Copy link
Member

amyblais commented Mar 7, 2019

Hi @ajdev6, letting you know that we've opened an improvement ticket for this here: https://mattermost.atlassian.net/browse/MM-14467. There is no need to open a feature request issue as we can track this via the Jira ticket. Thanks for reporting this!

@enahum
Copy link
Contributor

enahum commented Mar 16, 2019

@ajdev6 can you help clarify what is missing in the payload?

@amyblais
Copy link
Member

Letting you know that the Jira ticket was closed since this is not supported yet.

Please share this idea on our feature request forum here: https://mattermost.uservoice.com/forums/306457-general.

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

No branches or pull requests

3 participants