Skip to content
This repository was archived by the owner on Mar 16, 2019. It is now read-only.

Fix Compilation Error in React Native 0.47.0 #452

Merged
merged 1 commit into from
Aug 2, 2017
Merged

Fix Compilation Error in React Native 0.47.0 #452

merged 1 commit into from
Aug 2, 2017

Conversation

Hizoul
Copy link
Contributor

@Hizoul Hizoul commented Aug 1, 2017

createJSModules was removedin React Native 0.47.0 and results in the attached Build Error.
removing @OverRide fixes build (didn't remove function because I don't know if it should be kept for downard compatability?)

node_modules/react-native-fetch-blob/android/src/main/java/com/RNFetchBlob/RNFetchBlobPackage.java:23: error: method does not override or implement a method from a supertype
    @Override
    ^
1 error

Incremental compilation of 1 classes completed in 0.219 secs.
:react-native-fetch-blob:compileReleaseJavaWithJavac FAILED

FAILURE: Build failed with an exception.

Thank you for making a pull request ! Just a gentle reminder :)

  1. If the PR is offering a feature please make the request to our "Feature Branch" 0.11.0
  2. Bug fix request to "Bug Fix Branch" 0.10.7
  3. Correct README.md can directly to master

createJSModules was removedin React Native 0.47.0 and results in the attached Build Error.
removing @OverRide fixes build (didn't remove function because I don't know if it should be kept for downard compatability?)

```
node_modules/react-native-fetch-blob/android/src/main/java/com/RNFetchBlob/RNFetchBlobPackage.java:23: error: method does not override or implement a method from a supertype
    @OverRide
    ^
1 error

Incremental compilation of 1 classes completed in 0.219 secs.
:react-native-fetch-blob:compileReleaseJavaWithJavac FAILED

FAILURE: Build failed with an exception.
```
@wkh237
Copy link
Owner

wkh237 commented Aug 2, 2017

Okay, thanks ! I think this is necessary, will see if that's backward compatible.

@wkh237 wkh237 merged commit d1d07d0 into wkh237:master Aug 2, 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.

2 participants