Skip to content
This repository has been archived by the owner on Sep 4, 2021. It is now read-only.

Change require to not access NativeModules directly, but rather via "… #1

Merged
merged 1 commit into from
Aug 9, 2015

Conversation

RoryCombe
Copy link
Contributor

…react-native" and make library compatible with React Native ^0.7.0

…react-native" and make library compatible with React Native ^0.7.0
@RoryCombe
Copy link
Contributor Author

Quite a few libraries have needed to implement this change with the new versions of React Native. I was using your library (thanks by the way) for older versions and now upgrading and encountered the errors listed here: facebook/react-native#1808

The way RNGeocoder is required in index.js seems to be all that needs to be changed.

devfd added a commit that referenced this pull request Aug 9, 2015
Change require to not access NativeModules directly, but rather via "…
@devfd devfd merged commit 2ffc331 into devfd:master Aug 9, 2015
@devfd
Copy link
Owner

devfd commented Aug 9, 2015

Hi @umfana

nice of you submitting a PR. Thanks

@RoryCombe RoryCombe deleted the patch branch August 12, 2015 15:47
orhan pushed a commit to orhan/react-native-geocoder that referenced this pull request Mar 21, 2017
Update headers for RN@0.40.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants