Skip to content

PaymentQueueWrapper: also implement shouldShowPriceConsent#1963

Merged
NachoSoto merged 1 commit into
mainfrom
wrapper-price-consent
Oct 7, 2022
Merged

PaymentQueueWrapper: also implement shouldShowPriceConsent#1963
NachoSoto merged 1 commit into
mainfrom
wrapper-price-consent

Conversation

@NachoSoto

@NachoSoto NachoSoto commented Sep 30, 2022

Copy link
Copy Markdown
Contributor

Follow up to #1962. This is also needed when StoreKit1Wrapper is not setup (see #1882).

@NachoSoto NachoSoto added the pr:fix A bug fix label Sep 30, 2022
@NachoSoto NachoSoto requested a review from a team September 30, 2022 19:39
@NachoSoto NachoSoto force-pushed the payment-queue-wrapper-transaction-observer branch from 688b5cf to 9d67a6c Compare September 30, 2022 19:43
@NachoSoto NachoSoto force-pushed the wrapper-price-consent branch from 8ec7553 to a020440 Compare September 30, 2022 19:44
@NachoSoto NachoSoto force-pushed the payment-queue-wrapper-transaction-observer branch from 9d67a6c to 8e43fe8 Compare September 30, 2022 20:42
@NachoSoto NachoSoto force-pushed the wrapper-price-consent branch from a020440 to f58db2d Compare September 30, 2022 20:42
@NachoSoto NachoSoto force-pushed the payment-queue-wrapper-transaction-observer branch from 8e43fe8 to 3ef87d2 Compare October 7, 2022 14:31
@NachoSoto NachoSoto force-pushed the wrapper-price-consent branch from f58db2d to 9e6649f Compare October 7, 2022 14:53
Base automatically changed from payment-queue-wrapper-transaction-observer to main October 7, 2022 14:54
Follow up to. This is also needed when `StoreKit1Wrapper` is not setup (see #1882).
@NachoSoto NachoSoto force-pushed the wrapper-price-consent branch from 9e6649f to 558033e Compare October 7, 2022 14:55
@NachoSoto NachoSoto merged commit 6785b6e into main Oct 7, 2022
@NachoSoto NachoSoto deleted the wrapper-price-consent branch October 7, 2022 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants