Skip to content

fix(twitter): Missing com.twitter.sdk.android #507

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

Merged
merged 1 commit into from
Jun 9, 2023

Conversation

Ombuweb
Copy link
Contributor

@Ombuweb Ombuweb commented Jun 7, 2023

Fix the error: Could not find com.twitter.sdk.android

Fix the error: Could not find com.twitter.sdk.android
@cla-bot cla-bot bot added the cla: yes label Jun 7, 2023
@rigor789
Copy link
Member

rigor789 commented Jun 7, 2023

Is the sdk available on another repository? (Given jcenter is deprecated, and shouldn't be relied on anymore)

@Ombuweb
Copy link
Contributor Author

Ombuweb commented Jun 7, 2023

We tried adding the following repos and only jcenter() eliminated the error.

maven { url "https://jitpack.io/" }
jcenter()
mavenLocal()
        maven {
            url 'https://maven.google.com/'
            name 'Google'
        }
mavenLocal()

The library is found at mvnrepository.

Also, there is this Stack Overflow answer that says that the library is deprecated. Maybe we should replace it?

@rigor789
Copy link
Member

rigor789 commented Jun 7, 2023

Ah yeah, seems like it's a fairly outdated version of the sdk, ideally we could update to the new sdk package, but in the meantime, I guess having jcenter there is ok too.

The new sdk is called "com.twitter:twitter-api-java-sdk:1.1.4" (https://developer.twitter.com/en/docs/twitter-api/tools-and-libraries/sdks/overview)

@NathanWalker NathanWalker merged commit ee57c23 into NativeScript:main Jun 9, 2023
@NathanWalker
Copy link
Contributor

Thanks @Ombuweb available now in 1.0.1

@NathanWalker
Copy link
Contributor

We will plan to update to the more recent version for 2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants