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

support privacy config for clickToLoad #407

Merged
merged 2 commits into from
Jan 28, 2022

Conversation

ladamski
Copy link
Collaborator

@ladamski ladamski commented Jan 28, 2022

Task/Issue URL: https://app.asana.com/0/72649045549333/1201730253863268/f
Tech Design URL:
CC:

Description:
Support privacy config for ClickToLoad, requires these to land:
duckduckgo/BrowserServicesKit#56
duckduckgo/privacy-configuration#113

Currently this is failing because it needs clickToPlay to be added to the BSK branch in duckduckgo/BrowserServicesKit#56

Steps to test this PR:

  1. This PR will fail until add clickToPlay to privacyConfig BrowserServicesKit#56 lands. To test it, you can update the BSK package to point to that branch
  2. At which point it will compile but CTL will be disabled until we merge & deploy Enable ctl mac privacy-configuration#113
  3. Once that PR lands, CTL should be enabled again

You can also test with the local embedded config but you will need to disable the remotely fetched version which will override it (https://github.com/more-duckduckgo-org/macos-browser/blob/develop/DuckDuckGo/ContentBlocker/macos-config.json)

Testing checklist:

  • Test with Release configuration

Internal references:

Software Engineering Expectations
Technical Design Template
When ready for review, remember to post the PR in MM

@ladamski ladamski requested review from samsymons and tomasstrba and removed request for samsymons and tomasstrba January 28, 2022 18:35
@samsymons samsymons self-assigned this Jan 28, 2022
@ladamski ladamski force-pushed the la/support-ctl-privacy-config branch from 605d4c5 to 44f504f Compare January 28, 2022 19:00
Copy link
Collaborator

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

LGTM! Great work @ladamski!

@ladamski ladamski merged commit ece4c49 into develop Jan 28, 2022
@ladamski ladamski deleted the la/support-ctl-privacy-config branch January 28, 2022 19:51
samsymons added a commit that referenced this pull request Jan 31, 2022
# By Lucas Adamski (1) and others
# Via Tomas Strba
* develop:
  Version 0.18.5
  support privacy config for clickToLoad (#407)
  Automatically select available login (#405)

# Conflicts:
#	DuckDuckGo/SecureVault/Model/PasswordManagementItemListModel.swift
#	DuckDuckGo/SecureVault/View/PasswordManagementItemList.swift
#	DuckDuckGo/SecureVault/View/PasswordManagementViewController.swift
samsymons added a commit that referenced this pull request Feb 1, 2022
* develop:
  Version 0.18.5
  support privacy config for clickToLoad (#407)
  Automatically select available login (#405)
  initial FB Click to Load (WIP) (#329)
  onboarding updates (#398)
  Version 0.18.4
  Configuration of Sparkle - Setting SUAllowsAutomaticUpdates to NO (#404)
  Hide downloads button if the popover is opened/closed manually (#397)
samsymons added a commit that referenced this pull request Feb 18, 2022
# By Alexey Martemyanov (20) and others
# Via Tomas Strba (2) and others
* develop: (63 commits)
  Tweaks of suggestions and autocomplete (#403)
  Bump privacy dashboard to latest version (#409)
  Point to the latest BrowserServicesKit branch. (#414)
  Move embedded TDS from BSK to platform repo (#412)
  Image of shield with dot replaced (#410)
  Expand Fireproofing to include Local Storage and IndexedDB (#408)
  Version 0.18.5
  support privacy config for clickToLoad (#407)
  Automatically select available login (#405)
  initial FB Click to Load (WIP) (#329)
  onboarding updates (#398)
  Version 0.18.4
  Configuration of Sparkle - Setting SUAllowsAutomaticUpdates to NO (#404)
  Hide downloads button if the popover is opened/closed manually (#397)
  Textfield of the homepage is empty and unfocused right after switching to the homepage (#400)
  Remove navigatorCredentials (#392)
  Remove GPC header if it exists when not needed (#366)
  Version 0.18.3
  Fireproofing encrypted storage (#332)
  Fix Lock Screen UI issues (#399)
  ...

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo/Crash Reports/Model/CrashReportSender.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