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

Logging refactoring #1 Subscription and Content Blocking #3091

Merged
merged 11 commits into from
Aug 20, 2024

Conversation

federicocappelli
Copy link
Member

Task/Issue URL: https://app.asana.com/0/1205842942115003/1206750146033742/f
Tech Design URL: https://app.asana.com/0/1200194497630846/1206777133329590/f

Description:

Step 1 of the Logging refactoring

  • General structure
  • Subscription

Steps to test this PR:
Check subscription and content blocker logs are working

Definition of Done (Internal Only):


Internal references:

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

# Conflicts:
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
# Conflicts:
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
@federicocappelli federicocappelli marked this pull request as ready for review August 19, 2024 09:47
# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
federicocappelli added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Aug 20, 2024
Task/Issue URL:
https://app.asana.com/0/1205842942115003/1206750146033742/f
iOS PR: duckduckgo/iOS#3219
macOS PR: duckduckgo/macos-browser#3091
What kind of version bump will this require?: **Major**, breaking change
@federicocappelli federicocappelli merged commit 94409e4 into main Aug 20, 2024
17 of 18 checks passed
@federicocappelli federicocappelli deleted the fcappelli/oslog_to_logger branch August 20, 2024 13:27
samsymons added a commit that referenced this pull request Aug 25, 2024
…iption

* main: (25 commits)
  Bump version to 1.103.0 (249)
  Bump version to 1.103.0 (248)
  Resolving automatic update edge cases (#3142)
  PIR Time-Based Pixel: 24 Opt-Out Request Success Rate (#2942)
  Freemium Local Package and Freemium State Implementation (#3118)
  Bump rexml from 3.3.3 to 3.3.6 (#3141)
  Filter out inaccessible tasks from release (#3133)
  Bump rexml from 3.2.9 to 3.3.3 (#3139)
  Bump version to 1.103.0 (247)
  Specify secrets for pr.yml where called from another workflow (#3137)
  Fix secrets usage in CI workflows (#3134)
  Adding info about how we improve our products to the README.md (#3135)
  Add Freemium PIR Feature Flag (#3129)
  Release notes page loading fix + Adding close button to the upgrade notification (#3094)
  App Configuration app group (#3132)
  Use code signing in CI tests workflows (#3125)
  'track' rephrased to 'anonymously track' (#3121)
  Additional PIR opt out confirmation pixels (#3119)
  Logging refactoring #1 Subscription and Content Blocking (#3091)
  Update RemoteMessagingDebugMenu to work on a private-queue context (#3120)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants