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

Update dependencies and deprecated URL #9

Merged
merged 2 commits into from
Jun 26, 2021

Conversation

Mrhea
Copy link
Contributor

@Mrhea Mrhea commented Jun 25, 2021

Bintray is deprecated, so the app fails to run with a 403 error trying to get lazysodium. This PR fixes that issue and updates the project's dependencies.

Needs to be cleaned up still
Copy link

@rosasck rosasck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pending a small investigation on how to import lazy sodium

transitive = true
}

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this extra line

@@ -6,7 +6,7 @@ buildscript {
google()
jcenter()
mavenCentral()
maven { url "https://dl.bintray.com/terl/lazysodium-maven" }
maven { url "http://github.com/terl/lazysodium-android" }
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if theres a way to not import a github url! I left our conversation thinking about how it may be slow

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would this be better? https://mvnrepository.com/artifact/com.goterl.lazycode/lazysodium-java

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need lazysodium-android

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops ya, I just grabbed that from maven central as an example but probably would have used it. Thanks for catching that

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check if it works, not sure if it will i get a 404

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lazy sodium 5.x should be pulled from maven central without a custom import. But if it isn't working I will dig in at a later date because I am not supposed to be commenting today

dart_style:
dependency: transitive
description:
name: dart_style
url: "https://pub.dartlang.org"
source: hosted
version: "1.3.11"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you update the read me with the "pub get outdated" and "pub get dependencies"

}

implementation ('com.tozny.e3db:e3db-client-android:6.0.2@aar'){
implementation ('com.tozny.e3db:e3db-client-android:7.2.1@aar'){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be 7.2.2

@Mrhea Mrhea merged commit 315a57b into trunk Jun 26, 2021
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

Successfully merging this pull request may close these issues.

4 participants