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

add push notification #167

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

add push notification #167

wants to merge 7 commits into from

Conversation

mpettit
Copy link

@mpettit mpettit commented Dec 7, 2019

General strategy:

  1. On initial dashboard load, ask user for push notification permissions
  2. Save push notification token on user profile
  3. When we want to trigger notification, grab user token from db and send push notification by making POST request to Expo url

Good testing tool: https://expo.io/notifications

@mpettit mpettit changed the title add push notification - not fully working yet add push notification Dec 7, 2019
@mpettit
Copy link
Author

mpettit commented Dec 7, 2019

Eventually need to take a look at this for Android: https://docs.expo.io/versions/v35.0.0/guides/using-fcm/

@mpettit
Copy link
Author

mpettit commented Dec 7, 2019

TODO: re-enable save to user profile and get rid of warning

@doub1ejack
Copy link
Member

Hey Mike, thanks for taking this on. I'll look a little closer later, but thought I'd mention that I was playing with generating tokens earlier too. I suspect you're at or past the point I got to, but I made a draft pr with my little experiment for sharing: #170 (review)

@mpettit
Copy link
Author

mpettit commented Dec 18, 2019

Should unsubscribe from subscription using .remove()

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 this pull request may close these issues.

2 participants