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

Handle Android RN 0.47 breaking change #53

Closed
wants to merge 1 commit into from
Closed

Conversation

brunsy
Copy link

@brunsy brunsy commented Aug 2, 2017

The createJSModules method was removed from ReactPackage in version 0.47. Removing the override annotation will stop compiler errors in 0.47 while continuing to work in previous versions.

@ujwal-setlur
Copy link

Any ETA on merging this?

@edgareler
Copy link

It's really important to merge this, because of facebook/react-native@ce6fb33:

Breaking - remove unused registration of JS modules
Summary: It's now unnecessary to declare which JS modules you want to expose on your package. To upgrade, remove all overrides of createJSModules and keeping calling your JS modules as before.

@keshk26
Copy link

keshk26 commented Sep 29, 2017

Can you please merge this PR?

@brunsy
Copy link
Author

brunsy commented Nov 27, 2017

Closing as it looks like this was committed directly in by @Ehesp here e49cdb3

@brunsy brunsy closed this Nov 27, 2017
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.

4 participants