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

Billing library now deprecated/unusable #59

Open
PickleJesus123 opened this issue Oct 31, 2023 · 4 comments
Open

Billing library now deprecated/unusable #59

PickleJesus123 opened this issue Oct 31, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@PickleJesus123
Copy link

PickleJesus123 commented Oct 31, 2023

Godot version

4.1.2

System information

Android

Issue description

As of November 1st 2023 (tomorrow), Android Billing v4 will be entirely deprecated. The problem is, the master branch specifically targets this deprecated v4. However, there is another branch (billing-v5) which is up to date and working great, and it should become the new 'master' branch. Other users have brought this up, but it seems that no maintainers have been checking in! #51 (comment)
billing

I would be happy to submit a PR that completely updates the billing plugin documentation page, but the plugin itself should probably be brought up to date with the billing-v5 branch first!

Steps to reproduce

N/A

Minimal reproduction project

N/A

@akien-mga akien-mga added the bug Something isn't working label Oct 31, 2023
@akien-mga akien-mga transferred this issue from godotengine/godot Oct 31, 2023
@akien-mga
Copy link
Collaborator

Transferred to https://github.com/godotengine/godot-google-play-billing as it's not an issue with Godot itself.

AFAIK the current master branch is compatible with Google Play Billing 5, there was just a bug in #45 where the version had been mistyped. It should now be fixed by #53, bumping it to 5.2.1 for compatibility with Android 14.

The billing-v5 branch used for #36 seems to be a bigger refactor to make use of the new APIs which was never completed. There's also #52 building on top of it. But there's no active maintainer for this repo so it's difficult to get those reviewed, merged and released.

Can you confirm whether the current master branch is suitable to fulfill Google Play's requirements?

@akien-mga
Copy link
Collaborator

I'm not able to test it myself, but I made a new 1.2.0 release from the master branch that targets Google Play Billing 5.2.1: https://github.com/godotengine/godot-google-play-billing/releases/tag/1.2.0

Please let me know if it's working.

@PickleJesus123
Copy link
Author

PickleJesus123 commented Nov 1, 2023

Transferred to https://github.com/godotengine/godot-google-play-billing as it's not an issue with Godot itself.

AFAIK the current master branch is compatible with Google Play Billing 5, there was just a bug in #45 where the version had been mistyped. It should now be fixed by #53, bumping it to 5.2.1 for compatibility with Android 14.

The billing-v5 branch used for #36 seems to be a bigger refactor to make use of the new APIs which was never completed. There's also #52 building on top of it. But there's no active maintainer for this repo so it's difficult to get those reviewed, merged and released.

Can you confirm whether the current master branch is suitable to fulfill Google Play's requirements?

@akien-mga

The master branch is using deprecated methods (querySkuDetailsAsync(), etc)

Taken from my subscription in the Google Play console:
deprecated

And a code snippet from the master branch that uses these deprecated methods (in GodotGooglePlayBilling.java)
deprecated2

They expect us to use queryProductDetailsAsync() now, among other methods (all implemented in billing-v5 branch)

Also, it seems like the Godot Billing Documentation currently uses the methods/callbacks from billing-v5 branch, not the methods/callbacks used in master. I am using the billing-v5 branch, and other than this bug, it seems to be working flawlessly for my subscriptions

@kyadalu1
Copy link

kyadalu1 commented Jan 6, 2024

Try this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants