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 Android IAP in C# project to Godot 4.3 #1104

Merged
merged 3 commits into from
Aug 23, 2024

Conversation

JavierStark
Copy link
Contributor

@JavierStark JavierStark commented Aug 22, 2024

Attempt to update Android IAP in C# project to version 4.3 which was marked as needing an update in [#697]

Changes:

  • Dotnet target framework net472 -> net6.0
  • Correct use of Variants with AsType()
  • Signal connection using SignalName and callable creation with Callable.From()

@JavierStark JavierStark changed the title Update project to 4.3 Update Android IAP in C# project to Godot 4.3 Aug 22, 2024
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

Thanks! Congratulations for your first merged pull request 🎉

mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
mono/android_iap/Main.cs Outdated Show resolved Hide resolved
@Calinou Calinou merged commit 13b2787 into godotengine:master Aug 23, 2024
1 check passed
@JavierStark
Copy link
Contributor Author

I found some problems when building for Android and after uploading into Google Play Store. I will fix it shortly

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

Successfully merging this pull request may close these issues.

2 participants