Thank you for showing interest in contributing to React-Native-Navigation! This project is developed and maintained by Wix in collaboration with the community.
There are various ways in which you can contribute to this library, not all require knowledge or expertise in native development. Listed below is all you need to get started with your first contribution.
Stack Overflow is used by developers to find answers and troubleshoot code. Users are encouraged to post questions on SO and tag them with the wix-react-native-navigation tag. Answering issues on SO is very helpful and beneficial to the community, only this time, there's a personal angle - you can boost your SO ranking and rack up points quickly.
Another popular communication channel is our Discord channel where users post questions and consult with each other. You're welcome to join the discussions and answer questions. This is also a great place to meet other community members and project maintainers.
There are 2 ways to test your contribution to this project on iOS:
- You can contribute and test your changes using the playground provided.
- You can install React-Native-Navigation in a new or existing project and set your project to use your local clone of React-Native-Navigation in the podfile. The following steps are required in order to make the project work with your local environment:
Install React-Native-Navigation as you'll usually do in your project, using
npm install --save react-native-navigation.
Add the following configuration to the
react-native.config.js file, at the root of your project.
Clone the project using
git clone https://github.com/wix/react-native-navigation at your desired location.
Add the following line in the Podfile:
You're done. Happy coding!
Found a bug? Missing a feature? Go ahead and open an issue! Make sure to add all details mentioned in the issue template. If you're interested in suggesting a big change, please speak to one of the admins on Discord to coordinate your effort.
Although the issue tracker is used solely for bug reports, issues are frequently opened for questions or to request assistance. As the project grows in popularity, more issues remain unanswered for long periods of time.
Some issues can be trivial and easy to pinpoint - a missing import statement or apostrophe, wrong layout structure, etc. If you're an experienced user, helping out newcomers can be quite satisfying and allows maintainers to focus on features and bug fixes.
Some issues are tagged as "looking for contributors". These issues have been recognized by the team, but aren't prioritized by Wix developers due to lack of time or for some other reason. We leave these issues to our community and you're more than welcome to take a crack at them. If you'd like to submit a PR, see these instructions on how to run the project locally.
Reproducing bugs takes time. Lots of time. Usually that's because an issue is lacking important information, which then causes lots of back and forth communication between maintainers and users. Help us address more bugs and issues by providing reproductions.
Providing reproductions improves the chances of an issue being prioritized by maintainers!
If an issue is reproduced with a specific combination of options, posting these options will usually suffice. If a specific layout structure is involved or specific actions need to be performed in a certain order - then we ask that you fork the project and push a branch with a reproduction to the Playground app.
Check out the list of issues requiring reproductions.
So you've fixed a bug or added a feature and you're ready to submit a pull request 🎉🎊 Make sure the title is clear and describes the reason for the PR. Please include any information you can which will help us understand the reasons for the PR, what it fixes and what it changes. Please include before/after pictures/gifs when appropriate.
This project is driven by tests. Before implementing any feature or fixing any bug, a failing test (e2e or unit or both) should be added, depending on the environment of where the fix should be implemented. For example, for an API change, a failing e2e should be written. For a small bug fix in Android, for example, a unit test in Android should be added.
This will ensure good quality throughout the life of the project and will help avoid unexpected breakages.
No PR will be accepted without adequate test coverage.
If you need help running the project, have a look at the Playground app section. You can run the tests using the scripts below.
Besides an overview of basic React Native Navigation functionality, the Playground app can (and rather should) be used for e2e tests and reproductions. If your change requires an e2e, add it to the playground app, to the appropriate screen. Again, quick setup instructions available in Playground app section of these docs.
If a screen contains too many buttons, e2e tests might fail if the button is positioned out of screen bounds because Detox matchers detect it's invisible.
A pre-commit hook will verify that the linter passes when committing.
|The project itself composed of:
|android sources and unit tests
|iOS sources and unit tests
|TypeScript sources and unit tests
|the entry point for
import Navigation from 'react-native-navigation'
|detox e2e tests on both Android and iOS
|The end-user project all e2e tests run against. Contains its own
ios. Does not have its own package.json, depends on the local
<root>/lib for faster local development (no need to
npm install locally).
npm run build
|compiles TypeScript sources
npm run clean
|cleans all build directories, stops packager, fixes flakiness by removing watchman cache, etc.
npm run start
|starts the react-native packager for local debugging
npm run xcode
|for convenience, opens xcode in this project
npm run install-android
|builds playground debug/release version and installs on running android devices/emulators.
npm run uninstall-android
|uninstalls playground from running android devices/simulators
npm run test-js
npm run test-unit-ios
|runs ios unit tests in debug/release
npm run test-unit-android
|runs android unit tests in debug/release
npm run test-e2e-ios
|runs the ios e2e tests using detox in debug/release
npm run test-e2e-android
|runs the android e2e tests using detox in debug/release
npm run test-all
|runs all tests in parallel