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

Feature/verify receipt auto refresh #213

Merged
merged 9 commits into from
May 18, 2017

Conversation

bizz84
Copy link
Owner

@bizz84 bizz84 commented May 18, 2017

This PR simplifies the receipt verification flows by removing the refreshReceipt method from the public API.

Now clients only need to call verifyReceipt and the receipt is refreshed internally if needed.

Related issue: #42

/**
* Verify application receipt
* - Parameter password: Only used for receipts that contain auto-renewable subscriptions. Your app’s shared secret (a hexadecimal string).
* - Parameter session: the session used to make remote call.
Copy link
Owner Author

Choose a reason for hiding this comment

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

Out of date documentation

@@ -30,13 +30,14 @@ public class SwiftyStoreKit {

private let paymentQueueController: PaymentQueueController

private var receiptRefreshRequest: InAppReceiptRefreshRequest?
fileprivate let receiptVerificator: InAppReceiptVerificator

init(productsInfoController: ProductsInfoController = ProductsInfoController(),
paymentQueueController: PaymentQueueController = PaymentQueueController(paymentQueue: SKPaymentQueue.default())) {

Copy link
Owner Author

Choose a reason for hiding this comment

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

pass receiptVerificator to init

@bizz84 bizz84 merged commit 85ac45c into develop May 18, 2017
@bizz84 bizz84 deleted the feature/verify-receipt-auto-refresh branch May 18, 2017 16:08
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.

None yet

1 participant