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

Remove isSubscriptionEnabled check when attempting to delete NetP token #701

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

quanganhdo
Copy link
Member

@quanganhdo quanganhdo commented Mar 6, 2024

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/1203137811378537/1206780414627528/f
iOS PR: duckduckgo/iOS#2548
macOS PR: duckduckgo/macos-browser#2332
What kind of version bump will this require?: Patch

Optional:

Tech Design URL:
CC:

Description:

Steps to test this PR:
1.
1.

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16
  • macOS 10.15
  • macOS 11
  • macOS 12

Internal references:

Software Engineering Expectations
Technical Design Template

@quanganhdo quanganhdo self-assigned this Mar 6, 2024
@quanganhdo quanganhdo marked this pull request as ready for review March 6, 2024 21:56
@quanganhdo quanganhdo merged commit 5b354c6 into main Mar 7, 2024
12 checks passed
@quanganhdo quanganhdo deleted the anh/netp-delete-token-fix branch March 7, 2024 01:18
diegoreymendez pushed a commit that referenced this pull request Mar 7, 2024
* Remove isSubscriptionEnabled check when attempting to delete NetP token (#701)

* Bump C-S-S to 5.4.0

---------

Co-authored-by: Anh Do <[email protected]>
samsymons added a commit that referenced this pull request Mar 8, 2024
* main:
  Changes to 114.1.0-1 hotfix branch (#706)
  Navigation: Improve same-document navigation handling (#702)
  Add pixels for our main VPN funnels (#698)
  Remove Autoconsent subfeature (#697)
  #URL macro (#657)
  Handle expired entitlement in NetP (#692)
  Remove isSubscriptionEnabled check when attempting to delete NetP token (#701)
  Remove CGNAT range. (#691)
  Subscription keychain sharing for access token (#690)
  Integrate confirm entitlements endpoint (#700)
  fix bundle name in breakByRaisingSigInt (#699)
  Adds support for deleting all logins (#672)
  Bump C-S-S to 5.2.0 (#695)
  Update xcode-version to 15.2. (#688)
  Bump Tests/BrowserServicesKitTests/Resources/privacy-reference-tests (#687)
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