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

Pull notifications - Android #386

Open
nghiacc opened this issue Feb 28, 2021 · 0 comments · May be fixed by #396
Open

Pull notifications - Android #386

nghiacc opened this issue Feb 28, 2021 · 0 comments · May be fixed by #396
Assignees

Comments

@nghiacc
Copy link
Collaborator

nghiacc commented Feb 28, 2021

Purpose: With current mobile platform's push notifications, all notification messages must go through the platform service (Apple Push Notification service - APNs for iOS or Firebase Cloud Messaging - FCM) and centrally managed via a remote push notification server. This represents a challenge for a decentralised application like Stamp.

While no perfect technologies are yet available to offer a decentralised notification delivery, Stamp will initially adopt a pull notification technique which leverage background tasks to query the server periodically and update the user if there are new messages available for them.

With Android, the background task technique appears to work relatively well. however there is a noticeable performance hit in battery life, which needs to be mitigated and/or inform users about. The notification speed is also not as fast as the push notification, depending on the background task schedule.

A/C:

  • The notification must be able to notify users within 5 minutes of successful broadcast
  • There is no significant battery drain to device
  • There is no significant impact to device performance
  • There should be a message to inform users about the expected notification speed and the decrease in performance/battery (if noticeable)
@vince8x vince8x linked a pull request May 2, 2021 that will close this issue
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

Successfully merging a pull request may close this issue.

2 participants