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.

Sign up for GitHub

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

Jump to bottom

Polling notification #396

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Polling notification #396

wants to merge 23 commits into from

Conversation

vince8x
Copy link
Collaborator

@vince8x vince8x commented May 2, 2021

This is PR for polling new messages from the server periodically in the background and notify locally on the device
The interval for android device is 3 minutes and for ios is 15 minutes, but for the ios we cannot control when the polling run (it all depends on the device, maybe a day)

Some notes when implementing background polling:

  • axios does not work on the background, need to use native http
  • leveldb does not work on the background, need to save background new messages in the vuex store independently from the leveldb and save lastServerTime in capacitor Storage

viet and others added 21 commits March 17, 2021 09:04
Add new package for background fetch cordova-plugin-background-fetch
898efc7
@vince8x
Add ios Capabilities and permission for background task id
6976cba
Add background polling to check new messages and notify locally
63d0171
@vince8x
Change to use local notification instead of in-app notification for n…
80a6548 
…ew messages
@vince8x
Rollback temporary code to disable web socket
c054a8b
@vince8x
Request local notification permission on ready
99cd757
@vince8x
Fix polling local notification for ios
41c6c2a
Use different polling interval for android
43c25d6
@vince8x
Mobile local notification when receiving message via websocket
cd17b0d
@vince8x
Fix incorrect syntax notification
b7d8321
@vince8x
Process the messages after running the background fetch new messages
27e9e31
Refactor notify has new messages
438be95
Add request to ignore battery optimization
bf0060f
Fix has new messages action
997052e
@vince8x
Fix save storage in background mode
1476785
@vince8x
Use native http
2668939
@vince8x
Fix getMessages promise and remove console log
73b0fa5
@vince8x
Change polling interval to 3 minute on android
77a3e95
Store and process background message in memory
ad304a3
Merge branch 'master' into fb-polling-notification
51fef3e 
# Conflicts:
#	package.json
#	src/adapters/vuex-relay-adapter.js
#	src/cashweb/relay/index.js
#	src/store/modules/chats.js
#	src/utils/notifications.js
#	yarn.lock
@vince8x
Change async code to process in background messages
45a72ea
@vince8x vince8x linked an issue May 2, 2021 that may be closed by this pull request
Pull notifications - Android #386
Open
@vince8x vince8x requested a review from schancel May 2, 2021 04:54
@schancel schancel force-pushed the master branch 2 times, most recently from 92bc00d to 5090c9f Compare September 9, 2021 04:44
@schancel schancel force-pushed the master branch 2 times, most recently from 2babc5c to 007671d Compare March 20, 2022 01:57
@schancel schancel force-pushed the master branch 2 times, most recently from 1c1cdf5 to 3938e1b Compare September 4, 2022 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@schancel schancel Awaiting requested review from schancel

Assignees

@schancel schancel

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Pull notifications - Android
2 participants
@vince8x @schancel