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
BSK support for FB CTL update #760
base: main
Are you sure you want to change the base?
Conversation
0402bab
to
a4ec422
Compare
@@ -221,7 +221,7 @@ fileprivate extension KnownTracker.Rule { | |||
private extension KnownTracker.ActionType { | |||
|
|||
func toTrackerResolverRuleAction() -> TrackerResolver.RuleAction { | |||
self == .block ? .blockRequest : .allowRequest | |||
self == .block || self == .blockCtlFB ? .blockRequest : .allowRequest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about line 209 above? We might also need to address the custom action here?
20e59fa
to
717db16
Compare
This PR has been inactive for more than 7 days and will be automatically closed 7 days from now. |
This PR has been closed after 14 days of inactivity. Feel free to reopen it if you plan to continue working on it or have further discussions. |
Please review the release process for BrowserServicesKit here.
Required:
Task/Issue URL: https://app.asana.com/0/72649045549333/1205105078450227/f
iOS PR: TBD (hopefully N/A)
macOS PR: https://github.com/duckduckgo/macos-browser/pull/2162
What kind of version bump will this require?: Major/Minor/Patch
Optional:
Tech Design URL: https://app.asana.com/0/1201720254973470/1206422390411022/f
CC:
Description:
Updates BSK to support separate clickToLoad rule list, with associated surrogates changes. CTL is disabled for iOS
Steps to test this PR:
1.
1.
OS Testing:
Internal references:
Software Engineering Expectations
Technical Design Template