Skip to content

Fix Billing Client initialization & Update SDK integration guide in README#90

Merged
HakimHauston merged 1 commit into
mainfrom
fix-billing-client-and-readme
Jun 25, 2026
Merged

Fix Billing Client initialization & Update SDK integration guide in README#90
HakimHauston merged 1 commit into
mainfrom
fix-billing-client-and-readme

Conversation

@sotash

@sotash sotash commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

This PR fixes a compiler/runtime issue with PlayPcSdkFactory.CreateBillingClient by passing the required BillingClientParams configuration, satisfying the updated SDK signature. It also updates the integration instructions in README.md to reflect the new tarball-based package installation workflow.


Changes

1. Code Updates

  • GameStoreManager.cs:
    • Fixed the initialization of _billingClient to match the new SDK factory signature.
    • Created and passed a BillingClientParams object with EnablePendingPurchases = true to PlayPcSdkFactory.CreateBillingClient(clientParams).

2. Documentation Updates

  • README.md:
    • Updated Step 2 to reflect the new SDK installation method via the Unity Package Manager (installing the .tgz package from a tarball).
    • Refreshed the download URL, authorship info, and modification date to match the current guide structure.

… guide in README

- Update [GameStoreManager.cs](file:///c:/projects/repo/native-pc-sdk-sample/sdk-client-unity/Assets/Scripts/GameStoreManager.cs) to initialize `BillingClient` with `BillingClientParams` enabling pending purchases (`EnablePendingPurchases = true`).
- Update [README.md](file:///c:/projects/repo/native-pc-sdk-sample/sdk-client-unity/README.md) to reflect the new SDK installation workflow via the Unity Package Manager (tarball `.tgz` package) instead of dragging and dropping a `.unitypackage` file.
- Update authorship info, refresh the last updated date, and point the download URL to the official developer documentation page.
@sotash sotash self-assigned this Jun 25, 2026
@HakimHauston HakimHauston merged commit 8311c47 into main Jun 25, 2026
6 checks passed
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.

2 participants