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

Break the Subscription module’s dependency on BSK #2721

Merged
merged 5 commits into from
May 1, 2024

Conversation

samsymons
Copy link
Collaborator

Task/Issue URL: https://app.asana.com/0/1199230911884351/1207201481522088/f
Tech Design URL:
CC:

Description:

Client PR for duckduckgo/BrowserServicesKit#802.

Steps to test this PR:

  1. Test that Privacy Pro and the VPN works (test that you can buy a subscription and connect)

Internal references:

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

@samsymons samsymons requested a review from quanganhdo April 30, 2024 05:01
samsymons added 2 commits May 1, 2024 15:52
# By Fernando Bunn (3) and others
# Via GitHub
* main:
  Revert "Use NotLocalizedString on DBP error screen copy" (#2731)
  Use NotLocalizedString on DBP error screen copy
  Bump version to 1.86.0 (180)
  Bump version to 1.86.0 (179)
  Update embedded files
  Tentative fix for tab crashes (#2727)
  macOS: Set Error Reporter on All SecureVault Initializations  (#2717)
  Bump version to 1.86.0 (178)
  Alessandro/create subtask on pr requested (#2692)
  Prevent queued operations from cancelling manual scans (#2719)
  Fix Tab not closed for redrected download (#2715)
  Egress Server Failure Detection & Recovery (#2657)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
samsymons added a commit to duckduckgo/BrowserServicesKit that referenced this pull request May 1, 2024
Task/Issue URL: https://app.asana.com/0/1199230911884351/1207201481522088/f
iOS PR: duckduckgo/iOS#2804
macOS PR: duckduckgo/macos-browser#2721
What kind of version bump will this require?: Major

Description:

This PR removes the BSK dependency from the Subscription library.
@samsymons samsymons merged commit 401b0e9 into main May 1, 2024
19 checks passed
@samsymons samsymons deleted the sam/remove-subscription-bsk-dependency branch May 1, 2024 23:09
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