Skip to content

Added tests for PurchasesOrchestrator invoking listenForTransactions only if SK2 is enabled#1618

Merged
NachoSoto merged 1 commit into
mainfrom
lazy-var-workaround-3
May 26, 2022
Merged

Added tests for PurchasesOrchestrator invoking listenForTransactions only if SK2 is enabled#1618
NachoSoto merged 1 commit into
mainfrom
lazy-var-workaround-3

Conversation

@NachoSoto

Copy link
Copy Markdown
Contributor

Follow up to #1593 and #1596.

@NachoSoto

NachoSoto commented May 26, 2022

Copy link
Copy Markdown
Contributor Author

This adds a fair amount of complexity, but I guess it's important to test.

@aboedo aboedo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

❤️ thanks for doing this!

Base automatically changed from lazy-var-workaround-2 to main May 26, 2022 21:59
@NachoSoto NachoSoto force-pushed the lazy-var-workaround-3 branch from acd5931 to 65aa452 Compare May 26, 2022 21:59
@NachoSoto NachoSoto merged commit 5a009f2 into main May 26, 2022
@NachoSoto NachoSoto deleted the lazy-var-workaround-3 branch May 26, 2022 22:00
@NachoSoto NachoSoto mentioned this pull request Jun 2, 2022
NachoSoto added a commit that referenced this pull request Jun 2, 2022
### New Features
* `Purchases.customerInfo()`: added overload with a new `CacheFetchPolicy` (#1608) via NachoSoto (@NachoSoto)
* `Storefront`: added `sk1CurrentStorefront` for Objective-C (#1614) via NachoSoto (@NachoSoto)

### Bug Fixes
* Fix for not being able to read receipts on watchOS (#1625) via Patrick Busch (@patrickbusch)

### Other Changes
* Added tests for `PurchasesOrchestrator` invoking `listenForTransactions` only if SK2 is enabled (#1618) via NachoSoto (@NachoSoto)
* `PurchasesOrchestrator`: removed `lazy` hack for properties with `@available` (#1596) via NachoSoto (@NachoSoto)
* `PurchasesOrchestrator.purchase(sk2Product:promotionalOffer:)`: simplified implementation with new operator (#1602) via NachoSoto (@NachoSoto)
@aboedo aboedo mentioned this pull request Jun 3, 2022
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