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 BSK with Autofill changes #1086

Merged
merged 5 commits into from
Mar 29, 2022

Conversation

samsymons
Copy link
Contributor

@samsymons samsymons commented Mar 10, 2022

Task/Issue URL: https://app.asana.com/0/0/1201946516267418/f
Tech Design URL:
CC: @ayoy

Description:

This PR bumps the BSK commit to account for Autofill changes that are coming. None of these are used in iOS.

Steps to test this PR:

  1. Test that email autofill works. You can do this by signing in through Settings -> Email Protection, and then visiting a site that has an easy-to-test email field (like Next Draft).

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPad

OS Testing:

  • iOS 13
  • iOS 14
  • iOS 15

Internal references:

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

* develop:
  Update URL param ref tests (#1096)
  Downloads list + enhanced file handling  (#1095)
  Fix for the initial onboarding title not using the correct font (#1080)
  iOS Voice Search - Custom Dialog & Experiment (#1083)
  Bump app version to 7.65.13
  Improve local storage & cookie clearing by stopping all operations in open tabs (#1084)
@samsymons samsymons requested a review from ayoy March 25, 2022 05:34
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.

All good here! Tested with a physical device.

@ayoy ayoy self-assigned this Mar 28, 2022
samsymons added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Mar 29, 2022
Task/Issue URL: https://app.asana.com/0/0/1201938691568505/f
iOS PR: duckduckgo/iOS#1086
macOS PR: duckduckgo/macos-browser#461

Optional:

CC: @brindy

Description:

This PR updates BrowserServicesKit to handle the new pmHandlerStoreData call. As a part of this, it runs some logic to look up whether the provided payment method or address already exist in the database.

Steps to test this PR:

See macOS PR for more information
# By Michal Smaga (4) and others
# Via Michal Smaga (2) and GitHub (1)
* develop:
  Fix Xcode 13.3 compilation. (#1099)
  Update to BSK with a fix for percent encoding (#1091)
  Update the release notes
  Set version to 7.66.0
  Downloads UI tweaks (#1098)
  Fix for proper toolbar hiding when leaving Unprotected Sites view (#1097)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
@samsymons samsymons assigned samsymons and unassigned ayoy Mar 29, 2022
@samsymons samsymons merged commit 574a0f4 into develop Mar 29, 2022
@samsymons samsymons deleted the sam/update-bsk-for-autofill-cc-and-identity branch March 29, 2022 04:25
samsymons added a commit that referenced this pull request Apr 6, 2022
# By Michal Smaga (9) and others
# Via Jacek Łyp (2) and others
* develop: (21 commits)
  Update BSK with a fix for MacOS Website Breakage reporting (#1108)
  Improve swift style + swiftlint  (#1092)
  Update the release notes
  Set version to 7.66.1
  Fix for not sending errors inside ContentBlockerRulesManager (#1107)
  Use local file url instead of remote url for getting file size (#1106)
  Change the logic for tab preparation for data clearing (#1100)
  AMP code extraction into BSK (#1101)
  Fix incorrect m_download_tried_to_present_preview_without_tab pixel reporting (#1105)
  Fix AR previews not working on iOS 15.4 (#1104)
  Update BSK with Autofill changes (#1086)
  Fix Xcode 13.3 compilation. (#1099)
  Update to BSK with a fix for percent encoding (#1091)
  Update the release notes
  Set version to 7.66.0
  Downloads UI tweaks (#1098)
  Fix for proper toolbar hiding when leaving Unprotected Sites view (#1097)
  Update URL param ref tests (#1096)
  Downloads list + enhanced file handling  (#1095)
  Fix for the initial onboarding title not using the correct font (#1080)
  ...

# Conflicts:
#	Core/UserDefaultsPropertyWrapper.swift
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo/UserText.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