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

Update the Privacy Pro status attribute matcher #878

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

samsymons
Copy link
Contributor

@samsymons samsymons commented Jul 3, 2024

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/1199333091098016/1207731341550989/f
iOS PR: duckduckgo/iOS#3033
macOS PR: duckduckgo/macos-browser#2940
What kind of version bump will this require?: Patch (technically the public API has not changed)

Description:

This PR updates the Privacy Pro status attribute to match an array instead of a string.

Steps to test this PR:

  1. Check that tests pass

OS Testing:

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

Internal references:

Software Engineering Expectations
Technical Design Template

@samsymons samsymons requested a review from ayoy July 3, 2024 22:52
Copy link
Contributor

@ayoy ayoy left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@samsymons samsymons merged commit 6834243 into main Jul 4, 2024
9 checks passed
@samsymons samsymons deleted the sam/update-ppro-status-attribute branch July 4, 2024 08:46
samsymons added a commit that referenced this pull request Jul 7, 2024
# By Diego Rey Mendez (2) and others
# Via Diego Rey Mendez (1) and GitHub (1)
* main:
  Add connection tester failure pixels (#881)
  De-duplicate passwords on import (#869)
  Privacy Dashboard refactor (#879)
  Improve VPN logging logic (#877)
  Subscription refactoring #5 (#874)
  Fixes for Xcode16 (#864)
  Update the Privacy Pro status attribute to use an array. (#878)

# Conflicts:
#	Sources/NetworkProtection/PacketTunnelProvider.swift
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