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

Privacy Pro Free Trials - Models and API #3641

Merged
merged 4 commits into from
Dec 11, 2024

Conversation

aataraxiaa
Copy link
Contributor

@aataraxiaa aataraxiaa commented Dec 6, 2024

Task/Issue URL: https://app.asana.com/0/1208114992212396/1208796999534221/f
Tech Design URL: https://app.asana.com/0/481882893211075/1208773437150501/f

Description: Includes the following changes:

Steps to test this PR:

  1. Launch from Xcode
  2. Purchase a Privacy Pro Subscription via App Store and Staging
  3. Ensure purchase succeeds

Definition of Done (Internal Only):


Internal references:

Software Engineering Expectations
Technical Design Template
?


Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@aataraxiaa aataraxiaa self-assigned this Dec 6, 2024
@aataraxiaa aataraxiaa marked this pull request as ready for review December 6, 2024 16:01
@aataraxiaa aataraxiaa requested a review from miasma13 December 6, 2024 16:01
Copy link
Contributor

@miasma13 miasma13 left a comment

Choose a reason for hiding this comment

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

Works as expected, existing purchase flow unaffected. 👍

aataraxiaa added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Dec 11, 2024
Task/Issue URL:
https://app.asana.com/0/1208114992212396/1208796999534221/f
iOS PR: duckduckgo/iOS#3691
macOS PR: duckduckgo/macos-browser#3641
What kind of version bump will this require?: Minor

**Optional**:

Tech Design URL:
https://app.asana.com/0/481882893211075/1208773437150501/f

**Description**: This PR includes the following:

* Adds a new model type for Subscription Offers
* Adds a new API to `StorePurchaseManager` to retrieve Free Trial
Subscriptions
* Updates existing `subscriptionOptions` API to NOT return Free Trial
Subscriptions
* Adds abstractions on StoreKit types to enable testing
* Adds tests (**Note: Make sure to expand the
`StorePurchaseManagerTests` file when reviewing**)
@aataraxiaa aataraxiaa force-pushed the pete/pp-free-trials-offer-models branch from b6cd7f0 to ab5e6b1 Compare December 11, 2024 15:41
@aataraxiaa aataraxiaa merged commit 7854b22 into main Dec 11, 2024
20 checks passed
@aataraxiaa aataraxiaa deleted the pete/pp-free-trials-offer-models branch December 11, 2024 15:55
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