Skip to content

Conversation

plisitza
Copy link
Contributor

Summary

updated to reflect need for UNUserNotificationCenter. also brought README.md in line with markdown linter recommendations.

this is the result if my having struggled with it not working out of the box and stumbling on the solution, and a desire to help other avoid that.

Test Plan

I think this is not required for a readme change? other than to note that by following it my app now compiles and can push messages to a real device?

What's required for testing (prerequisites)?

see above?

What are the steps to reproduce (after prerequisites)?

a working app.

Compatibility

OS Implemented
iOS
Android

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I mentioned this change in CHANGELOG.md // I don't see a change log in the repo?
  • I updated the typed files (TS and Flow) // not applicable
  • I added a sample use of the API in the example project (example/App.js) // not applicable

@plisitza
Copy link
Contributor Author

sorry that took so long

Copy link
Collaborator

@Naturalclar Naturalclar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much! 🎉

@Naturalclar Naturalclar merged commit 9fbb9be into react-native-push-notification:master Feb 25, 2020
@capJavert
Copy link

@Naturalclar Hello, when can we see this published as new version on npm and pods?

@Naturalclar Naturalclar mentioned this pull request Feb 28, 2020
@Naturalclar
Copy link
Collaborator

@capJavert published as 1.0.7 🎉

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.

3 participants