-
-
Notifications
You must be signed in to change notification settings - Fork 105
feat: Add support for Expo push notifications #243
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
Conversation
Thanks for opening this pull request!
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you just add an entry to the README? At least like what was added in #239. If you think that there should be any additional information on how to configure the adapter, feel free to add it.
Yep, added a little snippet. |
@mortenmo Can you add test cases and ensure code is covered? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI failed, could you take a look?
… through PARSE_SERVER_CLEANUP_INVALID_INSTALLATIONS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good; is this ready for merge?
# [6.2.0](6.1.1...6.2.0) (2024-05-15) ### Features * Add support for Expo push notifications ([#243](#243)) ([8987b85](8987b85))
🎉 This change has been released in version 6.2.0 |
Expo for React Native is used heavily when creating mobile app. It comes with its own simplified way to create, manage and send push notifications.
https://docs.expo.dev/push-notifications/overview/
With the addition of Installations for Parse JS, it is now easier/possible to use the Parse Server send push notifications using internals
New Pull Request Checklist