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

[pull] develop from duckduckgo:develop #783

Open
wants to merge 1,795 commits into
base: develop
Choose a base branch
from

Conversation

pull[bot]
Copy link

@pull pull bot commented Jun 10, 2020

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull bot added ⤵️ pull merge-conflict Resolve conflicts manually labels Jun 10, 2020
amddg44 and others added 28 commits August 31, 2023 10:42
Task/Issue URL: https://app.asana.com/0/414709148257752/1204422898101808/f
Tech Design URL:
CC:

Description:
Modification of the Image Cache screen to display all cached favicons to accurately track that deleting bookmarks, closing tabs etc clear cached favicons correctly
Task/Issue URL: https://app.asana.com/0/0/1205381133187310/f

## Related PRs

BSK PR: duckduckgo/BrowserServicesKit#483
macOS PR: duckduckgo/macos-browser#1562

## Description

Integrates some of the latest changes from BSK.

--------

Co-authored-by: Sam Symons <[email protected]>
Task/Issue URL: https://app.asana.com/0/0/1205407743356198/f
Tech Design URL:
CC:

Description:

This PR bumps BSK to handle the removed incremental test feature.

This was never used by iOS, so this change should have no effect.
…1968)

Task/Issue URL: https://app.asana.com/0/30173902528854/1205306937274017/f

Description:
This is to deprecate current pixel because of a possibility to skew metrics in the future after a recent anomaly.
Task/Issue URL: https://app.asana.com/0/1177771139624306/1205398889304566/f
BSK PR: duckduckgo/BrowserServicesKit#490

Description:
Fixes an issue that causes passwords to be saved to local vault when Bitwarden is enabled.
Task/Issue URL: https://app.asana.com/0/414709148257752/1205415059499063
Tech Design URL:
CC:

Description:

Adds creation and modified dates to the keychain debug screen.

Steps to test this PR:

Run the app and create some keychain entries (autofill, email, etc)
Check the keychain debug screen shows the creation / modification dates
Task/Issue URL: https://app.asana.com/0/0/1205254652502303/f
Tech Design URL: https://app.asana.com/0/481882893211075/1205248329666560/f

Description:
Adding SyncSettingsAdapter to handle Sync for supported settings (currently only Email Protection auth status).
EmailManager notifications can now be posted from a background thread (by Sync) so their handlers
in the app were updated to run always on main thread (which is required because they update UI).
Sync is automatically triggered whenever Email Protection auth status change (unless the change was caused
by Sync itself).
Task/Issue URL: https://app.asana.com/0/0/1205416442924463/f

BSK PR: duckduckgo/BrowserServicesKit#491
macOS PR: duckduckgo/macos-browser#1588

## Description

Integrates some changes from BSK that make NetP now clear the auth token in the extension when `resetAllState` is called.
Also fixes a small bugfix with a callback not being called when it should.
Task/Issue URL: https://app.asana.com/0/414709148257752/1204144513391838/f
Tech Design URL:
CC:

Description:
Fixes an edge case where tapping on the search bar caused a crashed.
Task/Issue URL: https://app.asana.com/0/1203301625297703/1205419026232021/f

Description:
Update PR Checks workflow so that it searches JUnit test report for failed test cases, and reports every single
failed test case as new task in Asana (or a subtask in case of duplicates).
Task/Issue URL: https://app.asana.com/0/414235014887631/1205185501564329/f
Tech Design URL:
CC:

Description:
Removes Lock Screen widget pixels.

Steps to test this PR:

Run the app.
Add some Lock Screen widgets
Use them, check no pixels are sent related to their usage
Task/Issue URL: https://app.asana.com/0/1204099484721401/1202285063304466/f
BSK PR: duckduckgo/BrowserServicesKit#477

Description:
Updates the SecureVaultDelegate signature, so we can individually turn off autofill for cards, logins and identities on macOS (and on iOS later down the road).
…in screen (#2002)

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

Description:
Applies same authentication logic check to the details screen as to it's parent
Task/Issue URL: https://app.asana.com/0/414709148257752/1205255888342957/f
Tech Design URL:
CC:

Description:
Prevent autofill from autoprompting saved long in preview tabs i.e. when long pressing on a link
Task/Issue URL: https://app.asana.com/0/414235014887631/1205185500420499/f
Tech Design URL:
CC: @brindy

Description:

This PR fixes the iOS 17 widget view.

We're one beta away from GM, so I view it as unlikely that anything will improve here - to make sure we're safe, I've swapped over to a Circle view from Rectangle.
diegoreymendez and others added 28 commits November 27, 2023 18:18
Task/Issue URL: https://app.asana.com/0/0/1205949780297088/f
Tech Design URL: https://app.asana.com/0/481882893211075/1204986998781220/f

Description:
Add BookmarksFaviconFetcher that is used to fetch favicons for bookmarks received by Sync.
Fetcher is opt-in, controlled by a setting inside Sync settings (with an additional in-context onboarding
popup presented from client apps). Fetcher uses LinkPresentation framework to obtain a favicon
for a given domain, and in case of failure it falls back to checking hardcoded favicon URLs.
Fetcher keeps a state internally, by saving list of bookmarks IDs that need processing to a file on disk.
Fetcher plugs into clients' implementation of favicon storage by exposing FaviconStoring protocol.
Fetcher performs fetching on a serial operation queue. Each fetcher invocation cancels previously scheduled
operation and schedules a new one. Updating fetcher state is also scheduled on the operation queue -
state updates don't support cancelling and always finish before next operation is started.
Task/Issue URL: https://app.asana.com/0/414235014887631/1205953072928854/f
Tech Design URL:
CC:

Description:

New pixels for iOS toolbars project.

Address Bar

AddressBarSettingsPressed: New pixel m_addressbar_settings
AddressBarRefreshPressed: This is the m_r pixel and is only called from the address bar so already exists
AddressBarShieldPressed: This is the mp pixel so already exists
AddressBarSharePressed: This was one pixel with a parameter, but I'm splitting into two new ones for clarity:
new pixel: m_addressbar_share from the address bar
new pixel: m_browsingmenu_share from the menu
pixel removed: mb_sh has been removed
Share Sheet

Share result
New pixel m_sharesheet_result_success when success is true
New pixel m_sharesheet_result_fail when success is false
ShareSheetCopy: new pixel m_sharesheet_activity_copy
ShareSheetAddBookmark: new pixel m_sharesheet_activity_addbookmark
ShareSheetAddFavorite: new pixel m_sharesheet_activity_addfavorite
ShareSheetFindinPage: new pixel m_sharesheet_activity_findinpage
ShareSheetPrint: new pixel m_sharesheet_activity_print
ShareSheetAddToReadingList: new pixel m_sharesheet_activity_addtoreadinglist
Also includes adding app version by default to the pixels as part of privacy triage: https://app.asana.com/0/69071770703008/1206042861558143/f
Task/Issue URL: https://app.asana.com/0/414235014887631/1206073390175555/f
Tech Design URL:
CC:

Description:

This PR adds a row to the VPN waitlist view controller to show the T&C status, and correctly resets it when resetting all data.
Task/Issue URL: https://app.asana.com/0/0/1206056874665890/f

Description:

During the NetP iOS ship review, there was feedback that the server IP address labels should not feature the port. So now, rather than sending the host + port as the server address, I’m just sending the host.
Task/Issue URL: https://app.asana.com/0/0/1206027185389966/f
Tech Design URL:
CC:

Description:

This PR makes some final touches for the NetP waitlist, including:

Adding RMF support
Adding pixels
Updates NetP to only be visible to US users (unless they're internal)
Recording the date at which the user activated NetP, in order to show messages remotely if necessary
1. Introduce maestro test flows for address bar spoofing.
2. Update the omnibar text value to "about:blank" when a file download prompt is shown.
Task/Issue URL: https://app.asana.com/0/0/1205884549503153/f

Description:
Improve handling of lists in syncable models' payloads (which currently is limited to bookmarks
folders and favorites folders children). This change updates how lists are sent to Sync in PATCH
payloads by including current state as well as the set of removed and inserted items.
To allow for this, last sync payload is kept for all bookmarks folders in
lastChildrenPayloadReceivedFromSync property added on BookmarkEntity entity.
If the payload is not available (e.g. first sync), the legacy mechanism is used (i.e. sending only
the current contents of the list). The schema of a Sync response payload does not change so
the implementation handles it too.
Task/Issue URL: https://app.asana.com/0/1199230911884351/1206100276423377/f
Tech Design URL:
CC:

Description:

This PR updates BSK to include a NetP settings change. The change isn't used here in any way.
Task/Issue URL: https://app.asana.com/0/1205842942115003/1205692741026215/f
Tech Design URL: https://app.asana.com/0/1205842942115003/1205917507991804/f

**Description**:
This PR introduces a new version of BSK (PR: duckduckgo/BrowserServicesKit#578) that allows iOS and macOS to share the same web implementation of the `ReportBrokenSite` page and introduce improvements to Pixels and the broken site report data.

Additional changes
- The old native `Report broken site` page has been removed
- Added 2 new Pixels https://app.asana.com/0/1205842942115003/1205859726054342/f
- Removed legacy PrivacyDashboard.storyboard localised strings
@jaceklyp jaceklyp deleted the branch scope-demo:develop December 6, 2023 15:48
@jaceklyp jaceklyp deleted the develop branch December 6, 2023 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⤵️ pull merge-conflict Resolve conflicts manually
Projects
None yet
Development

Successfully merging this pull request may close these issues.