Update install instructions in README
See original GitHub issueFollowing the install instructions verbatim won’t work. react-native link
requires the library to be in your package.json
, and so it does nothing unless react-native-push-notification
is saved to the package.json
. To make things really clear, the instructions should be changed to:
npm install react-native-push-notification --save
react-native link
Issue Analytics
- State:
- Created 7 years ago
- Comments:5
Top Results From Across the Web
Keep Your README Install Instructions Up To Date - DockYard
A simple unit test ensures compatibility between your Elixir app's current version and the version constraint in your README's installation ...
Read more >Update Installation instructions in readme · Issue #24 - GitHub
Update the "Installation" section of readme.md: describe how to install the component using Add-ons Marketplace.
Read more >README.md template | Documenting your project - Drupal
Information about content and formatting of project README files. ... Installation (required, unless a separate INSTALL.md is provided) ...
Read more >Writing a README
Every GitHub repository should include a README.md file that provides any ... Complete setup/installation instructions: At this point, your setup and ...
Read more >Update the Readme | Talaria Software
In this article, we are talking about updating the readme file. When. Every time there is some new piece of information that every...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
@mcrowe there is actually already a pull request https://github.com/zo0r/react-native-push-notification/pull/326 . I actually don’t know why they wouldnt merge it…
Yup. Looks like this was fixed in commit e17d8a1f065dbe5f2f3ab864f0cf2e56bd778bc6, which is merged into master. Closing.