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 project #491

Closed

Conversation

erikrodriguez-se
Copy link
Contributor

Update android-browser-helper to modern SDKs, gradle versions, and dependencies and update code accordingly

Update android-browser-helper to modern SDKs, gradle versions, and dependencies and update code accordingly
Copy link
Collaborator

@orbital17 orbital17 left a comment

Choose a reason for hiding this comment

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

Thank you, Erik, that's amazing work!
I tried to revert the playbilling and also fixed several issues that affected the build in the other PR #492. Please take a look and let me know what you think.
If you need to make some changes, feel free to merge that branch into yours


implementation fileTree(dir: 'libs', include: ['*.jar'])
implementation 'com.android.billingclient:billing:6.2.1'
implementation 'org.chromium.net:cronet-embedded:119.6045.31'
implementation 'com.android.billingclient:billing:7.1.1'
Copy link
Collaborator

Choose a reason for hiding this comment

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

We can't update the play billing library from v6 to v7 yet, as there are other dependencies that need to be considered

@erikrodriguez-se
Copy link
Contributor Author

@orbital17 looks great! I've pulled the other branch and it looks good to me. If you want to merge that one and close this out that's fine with me

@ukaratkevich
Copy link
Collaborator

Closing in favor of #493.

@erikrodriguez-se thanks a lot for the PR, but we need to move forward with our own, because current may introduce regressions for future releases. As soon as we get #493 merged, we will proceed with #486 #488 #490.

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.

3 participants