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

TwitterSigninModule is not abstract and does not override abstract method onNewIntent #10

Closed
assem-ch opened this issue Oct 13, 2016 · 25 comments

Comments

@assem-ch
Copy link

React-native 0.35


/Users/job/workspace/spartask-mobile/node_modules/react-native-twitter-signin/android/src/main/java/com/goldenowl/twittersignin/TwitterSigninModule.java:32: error: TwitterSigninModule is not abstract and does not override abstract method onNewIntent(Intent) in ActivityEventListener
public class TwitterSigninModule extends ReactContextBaseJavaModule implements ActivityEventListener {
       ^
/Users/job/workspace/spartask-mobile/node_modules/react-native-twitter-signin/android/src/main/java/com/goldenowl/twittersignin/TwitterSigninModule.java:88: error: method does not override or implement a method from a supertype
    @Override
    ^
2 errors
:react-native-twitter-signin:compileReleaseJavaWithJavac FAILED
@jitu9717
Copy link

jitu9717 commented Oct 17, 2016

I am also getting this error version 0.31.0
/home/ttnd/projects/rally-bus/node_modules/react-native-twitter-signin/android/src/main/java/com/goldenowl/twittersignin/TwitterSigninModule.java:32: error: TwitterSigninModule is not abstract and does not override abstract method onNewIntent(Intent) in ActivityEventListener
public class TwitterSigninModule extends ReactContextBaseJavaModule implements ActivityEventListener {
^
1 error
:react-native-twitter-signin:compileReleaseJavaWithJavac FAILED

@jitu9717
Copy link

I resolved this by adding in file /node_modules/react-native-twitter-signin/android/src/main/java/com/goldenowl/twittersignin/TwitterSigninModule.java

public void onNewIntent(Intent intent) {
}

but now app crashes on clicking twitter icon

@RaviAsnani
Copy link

+1 Facing the same issue, any ideas on how to solve this?

@Rhernandez513
Copy link
Contributor

Rhernandez513 commented Oct 22, 2016

Also facing this issue, build log if it helps

react-native version 0.34

`Starting JS server...
Building and installing the app on the device (cd android && ./gradlew installDebug...
Incremental java compilation is an incubating feature.
Crashlytics was applied to an android-library project.
Android-library support is currently an incubating feature.
Contact support@fabric.io with any issues.
Crashlytics couldn't find the res folder specified in sourceSets.main.res, please make one. It picked: src/main/res
Crashlytics couldn't find the res folder specified in sourceSets.main.res, please make one. It picked: src/main/res

// Omitted a bunch of lines for brevity

:react-native-twitter-signin:preBuild UP-TO-DATE
:react-native-twitter-signin:preReleaseBuild UP-TO-DATE
:react-native-twitter-signin:checkReleaseManifest
:react-native-twitter-signin:preDebugAndroidTestBuild UP-TO-DATE
:react-native-twitter-signin:preDebugBuild UP-TO-DATE
:react-native-twitter-signin:preDebugUnitTestBuild UP-TO-DATE
:react-native-twitter-signin:preReleaseUnitTestBuild UP-TO-DATE
:react-native-twitter-signin:prepareComAndroidSupportAppcompatV72301Library UP-TO-DATE
:react-native-twitter-signin:prepareComAndroidSupportRecyclerviewV72301Library UP-TO-DATE
:react-native-twitter-signin:prepareComAndroidSupportSupportV42321Library UP-TO-DATE
:react-native-twitter-signin:prepareComDigitsSdkAndroidDigits1103Library UP-TO-DATE
:react-native-twitter-signin:prepareComFacebookFrescoDrawee0110Library UP-TO-DATE
:react-native-twitter-signin:prepareComFacebookFrescoFbcore0110Library UP-TO-DATE
:react-native-twitter-signin:prepareComFacebookFrescoFresco0110Library UP-TO-DATE
:react-native-twitter-signin:prepareComFacebookFrescoImagepipeline0110Library UP-TO-DATE
:react-native-twitter-signin:prepareComFacebookFrescoImagepipelineBase0110Library UP-TO-DATE
:react-native-twitter-signin:prepareComFacebookFrescoImagepipelineOkhttp30110Library UP-TO-DATE
:react-native-twitter-signin:prepareComFacebookReactReactNative0340Library UP-TO-DATE
:react-native-twitter-signin:prepareComFacebookSoloaderSoloader010Library UP-TO-DATE
:react-native-twitter-signin:prepareComTwitterSdkAndroidTweetComposer103Library UP-TO-DATE
:react-native-twitter-signin:prepareComTwitterSdkAndroidTweetUi1101Library UP-TO-DATE
:react-native-twitter-signin:prepareComTwitterSdkAndroidTwitter1131Library UP-TO-DATE
:react-native-twitter-signin:prepareComTwitterSdkAndroidTwitterCore166Library UP-TO-DATE
:react-native-twitter-signin:prepareIoFabricSdkAndroidFabric1310Library UP-TO-DATE
:react-native-twitter-signin:prepareOrgWebkitAndroidJscR174650Library UP-TO-DATE
:react-native-twitter-signin:prepareReleaseDependencies
:react-native-twitter-signin:compileReleaseAidl UP-TO-DATE
:react-native-twitter-signin:compileReleaseNdk UP-TO-DATE
:react-native-twitter-signin:compileLint UP-TO-DATE
:react-native-twitter-signin:copyReleaseLint UP-TO-DATE
:react-native-twitter-signin:compileReleaseRenderscript UP-TO-DATE
:react-native-twitter-signin:generateReleaseBuildConfig UP-TO-DATE
:react-native-twitter-signin:mergeReleaseShaders UP-TO-DATE
:react-native-twitter-signin:compileReleaseShaders UP-TO-DATE
:react-native-twitter-signin:generateReleaseAssets UP-TO-DATE
:react-native-twitter-signin:mergeReleaseAssets UP-TO-DATE
:react-native-twitter-signin:processReleaseManifest UP-TO-DATE
:react-native-twitter-signin:fabricGenerateResourcesRelease
Crashlytics could not parse the Android Project structure.
It looks like Crashlytics is being applied to your Android Library! Just remove
apply plugin: 'Crashlytics'
from your library project. If you're looking for Library support, please contact support@fabric.io
:react-native-twitter-signin:generateReleaseResValues UP-TO-DATE
:react-native-twitter-signin:generateReleaseResources
:react-native-twitter-signin:mergeReleaseResources UP-TO-DATE
:react-native-twitter-signin:processReleaseResources UP-TO-DATE
:react-native-twitter-signin:generateReleaseSources UP-TO-DATE
:react-native-twitter-signin:incrementalReleaseJavaCompilationSafeguard UP-TO-DATE
:react-native-twitter-signin:compileReleaseJavaWithJavac
:react-native-twitter-signin:compileReleaseJavaWithJavac - is not incremental (e.g. outputs have changed, no previous execution, etc.).
:react-native-twitter-signin:compileReleaseJavaWithJavac FAILED
/home/REDACTED/Software/REDACTED/REDACTED/node_modules/react-native-twitter-signin/android/src/main/java/com/goldenowl/twittersignin/TwitterSigninModule.java:32: error: TwitterSigninModule is not abstract and does not override abstract method onNewIntent(Intent) in ActivityEventListener
public class TwitterSigninModule extends ReactContextBaseJavaModule implements ActivityEventListener {
^
/home/REDACTED/Software/REDACTED/REDACTED/node_modules/react-native-twitter-signin/android/src/main/java/com/goldenowl/twittersignin/TwitterSigninModule.java:88: error: method does
not override or implement a method from a supertype
@OverRide
^
2 errors

FAILURE: Build failed with an exception.

  • What went wrong:
    Execution failed for task ':react-native-twitter-signin:compileReleaseJavaWithJavac'.

    Compilation failed; see the compiler error output for details.

  • Try:
    Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 11.955 secs
Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/android-setup.html
`

@duataud
Copy link

duataud commented Oct 31, 2016

Help. Same issue.

@khalidahmada
Copy link

i have same issue can can you help us please ?

@StephenMayeux
Copy link

I also have the same issue. How can we resolve this?

@duataud
Copy link

duataud commented Dec 1, 2016

This fork work for me https://github.com/drumtoad/react-native-twitter-signin.git

@Grundmanis
Copy link

React native 0.38.0 This commit helped me 04a56b5#diff-3dda93ff305afb460011e1627d3a275d

@cbartondock
Copy link

@duataud that fork also worked for me, except that I then see the following error

/Users/chrisdock/Desktop/Jdoe/jdoe-redux/android/app/src/main/java/com/jdoe/MainApplication.java:47: error: constructor TwitterSigninPackage in class TwitterSigninPackage cannot be applied to given types;
            new TwitterSigninPackage(),
            ^
  required: PermissionCallbackManager
  found: no arguments
  reason: actual and formal argument lists differ in length
1 error
:app:compileDebugJavaWithJavac FAILED

Googled this and couldn’t find anything… let me know if you have any ideas.

@cbartondock
Copy link

Edit: Switching to "github:rakannimer/react-native-twitter-signin” solved all the issues I was having on react 0.44.0

@eggybot
Copy link

eggybot commented May 22, 2017

Attention guys!

Add this to MainApplication.java
import com.goldenowl.twittersignin.TwitterSigninPackage;

And replace @GoldenOwlAsia package with RakanNimer .
(just use the android folder on the page in @rakannimer package)

Also take note the new login function code.

(thanks @cbartondock)

@franj0
Copy link

franj0 commented Jun 5, 2017

@eggybot
hi eggy

mainActivity
twitter_bug_2_720

MainApplication
twitter_bug_1

And I'm replaced android folder from RakanNimer package, but still not work
image

Plsss help, ty
email:eric.franjo@spark.ba

@franj0
Copy link

franj0 commented Jun 5, 2017

Sry, I solved this finally.
First time I linked manually,:

import com.goldenowl.twittersignin.TwitterSigninPackage;
new TwitterSigninPackage()

and not work, but with react native link it's working...
ty anyway

@franj0
Copy link

franj0 commented Jun 8, 2017

eggybot commented 16 days ago • edited
Attention guys!

Add this to MainApplication.java
import com.goldenowl.twittersignin.TwitterSigninPackage;

And replace @GoldenOwlAsia package with RakanNimer .
(just use the android folder on the page in @rakannimer package)

Also take note the new login function code.

(thanks @cbartondock)

Promise not working
image

Error: { NativeMap: {"email":"COULD_NOT_FETCH","userName":"franjo_eric","userID":"867700422731026432","name":"franjo_eric","authTokenSecret":"NOD3dwR9erA42LwLNDwYO7F3n6sjvPcqQZSnqvn60jCdd","authToken":"867700422731026432-jN35f9RrAg6BLbtT8NDAqdddibjtBhi"} }
at createErrorFromErrorData (NativeModules.js:115)
at NativeModules.js:78
at MessageQueue.__invokeCallback (MessageQueue.js:296)
at MessageQueue.js:120
at MessageQueue.__guard (MessageQueue.js:218)
at MessageQueue.invokeCallbackAndReturnFlushedQueue (MessageQueue.js:119)
at debuggerWorker.js:71

@eggybot
Copy link

eggybot commented Jun 9, 2017

that is the exact code I use but I didn't get that error, I tried connecting my twitter signin using firebase authentication for storing user data.

@rpandey66
Copy link

uncomment this method like this in TwitterSignInPackage and the issue will be solved

// @OverRide
// public List<Class<? extends JavaScriptModule>> createJSModules() {
// return Collections.emptyList();
// }

now it will work smoothly

Below is TwitterSigninPackage which looks like after commenting that method

package com.goldenowl.twittersignin;

import com.facebook.react.ReactPackage;
import com.facebook.react.bridge.JavaScriptModule;
import com.facebook.react.bridge.NativeModule;
import com.facebook.react.bridge.ReactApplicationContext;
import com.facebook.react.uimanager.ViewManager;

import java.util.Arrays;
import java.util.Collections;
import java.util.List;

public class TwitterSigninPackage implements ReactPackage {

@Override
public List<NativeModule> createNativeModules(ReactApplicationContext reactContext) {
    return Arrays.<NativeModule>asList(new TwitterSigninModule(reactContext));
}

// @OverRide
// public List<Class<? extends JavaScriptModule>> createJSModules() {
// return Collections.emptyList();
// }

@Override
public List<ViewManager> createViewManagers(ReactApplicationContext reactContext) {
    return Collections.emptyList();
}

}

@karta1502545
Copy link

I get the same issue here. Error: { NativeMap: {"email":"COULD_NOT_FETCH", Is there any solution?

@osama-ameer
Copy link

I am getting this error. Please help
Error: { NativeMap: {"email":"COULD_NOT_FETCH","userName":"JunaidQ37314523","userID":"1514560025057968128","name":"JunaidQ37314523","authTokenSecret":"mLRcaBemCenfU46RS3DDT8Yt9lgrd5GLJNVfQerEg06Xs","authToken":"1514560025057968128-jieWJKlYLzmyQFMWz3MxXk5nP1XIpN"} }

@mehraj43
Copy link

I am getting this error. Please help
Error: { NativeMap: {"email":"COULD_NOT_FETCH",

@mehraj43
Copy link

I am getting this error. Please help Error: { NativeMap: {"email":"COULD_NOT_FETCH","userName":"JunaidQ37314523","userID":"1514560025057968128","name":"JunaidQ37314523","authTokenSecret":"mLRcaBemCenfU46RS3DDT8Yt9lgrd5GLJNVfQerEg06Xs","authToken":"1514560025057968128-jieWJKlYLzmyQFMWz3MxXk5nP1XIpN"} }

hey did u resolve the issue if done can u tell me how to resolve it

@zabingosoftwares
Copy link

zabingosoftwares commented Aug 25, 2022

Please help Error: { NativeMap: {"email":"COULD_NOT_FETCH",

here is the same problem. did you get any solution ?can any one figure it out that it is library problem or twitter developer cosole settings problem? any solution is highly appriciated. thank you

@noeckl
Copy link

noeckl commented Sep 9, 2022

I figured out that this error does not prevent a successful login at least for firebase authentication. I posted more details on my method on Stack Overflow.

Could be that my solution also helps to solve the login problem without firebase. Potentially, simply parsing the error to retrieve the account information.

@mehraj43
Copy link

mehraj43 commented Sep 9, 2022 via email

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

No branches or pull requests