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

Factory-ize wrapper creation to allow React Native support #33

Merged
merged 1 commit into from
Apr 26, 2016

Conversation

dmaevac
Copy link
Contributor

@dmaevac dmaevac commented Apr 26, 2016

  • Refactor core to a factory function
  • Create factory consumers for react & react native
  • Update readme
  • Fixes React Native branch? #25

(Missing react native specific unit tests)

@coveralls
Copy link

coveralls commented Apr 26, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 143b198 on DveMac:master into 366335c on mjrussell:master.

@mjrussell
Copy link
Owner

@DveMac thanks this is great!

Are you using this with your react native app? Im curious how well it plays with https://github.com/aksonov/react-native-router-flux and if I should make an example since react-router doesn't support native

@mjrussell mjrussell merged commit 69d7cee into mjrussell:master Apr 26, 2016
This was referenced Apr 26, 2016
@dmaevac
Copy link
Contributor Author

dmaevac commented Apr 26, 2016

Im currently using react-router-redux and react-router (not a react native specific apdater). React router I can pass my own history object and therefore use almost the entirely same routing setup across react and react native versions of the app. Im not totally convinced its the way to go, but its where Im at right now!

@mjrussell
Copy link
Owner

Thanks for the info, I was planning on playing with my first React Native app so helpful to hear that the custom history works well in native.

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