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

macOS in-context signup updates #359

Merged
merged 17 commits into from
Jun 9, 2023
Merged

Conversation

alistairjcbrown
Copy link
Member

@alistairjcbrown alistairjcbrown commented May 19, 2023

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/0/1204636039042602/f
iOS PR: duckduckgo/iOS#1756
macOS PR: duckduckgo/macos-browser#1209
What kind of version bump will this require?: Major -- a new parameter is added to ContentScopeFeatureToggles

Optional:

Tech Design URL:
CC: @amddg44 @afterxleep

Description:
Adds support for getting and setting in-context signup dismissal, as well as opening and closing the Email Protection web app tab for users signing-in/signiing-up

Steps to test this PR:
1.
1.

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16
  • macOS 10.15
  • macOS 11
  • macOS 12

Internal references:

Software Engineering Expectations
Technical Design Template

Package.swift Outdated
@@ -26,7 +26,7 @@ let package = Package(
.library(name: "Navigation", targets: ["Navigation"]),
],
dependencies: [
.package(url: "https://github.com/duckduckgo/duckduckgo-autofill.git", exact: "7.0.1"),
.package(url: "https://github.com/duckduckgo/duckduckgo-autofill.git", revision: "2fa8222"),
Copy link
Member Author

Choose a reason for hiding this comment

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

Pointing to in-review branch -- duckduckgo/duckduckgo-autofill#317
This will be updated before merging to an autofill release

@alistairjcbrown alistairjcbrown force-pushed the daniel/macos-incontext-signup-updates branch 2 times, most recently from d80b4c0 to 61fc43b Compare May 21, 2023 20:49
@alistairjcbrown alistairjcbrown force-pushed the daniel/macos-incontext-signup-updates branch 4 times, most recently from 850fa8e to 74728c8 Compare May 25, 2023 12:45
@alistairjcbrown alistairjcbrown requested review from a team, jaceklyp and jotaemepereira and removed request for a team and jaceklyp May 25, 2023 13:59
@afterxleep afterxleep closed this May 26, 2023
@afterxleep afterxleep deleted the daniel/macos-incontext-signup-updates branch May 26, 2023 10:11
@alistairjcbrown alistairjcbrown restored the daniel/macos-incontext-signup-updates branch May 26, 2023 16:23
@alistairjcbrown
Copy link
Member Author

@afterxleep Ooops, looks like this branch got deleted. Want me to rename it under abrown/ or are we ok keeping it about for code review?

Copy link
Contributor

@jotaemepereira jotaemepereira left a comment

Choose a reason for hiding this comment

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

LGTM

@alistairjcbrown alistairjcbrown force-pushed the daniel/macos-incontext-signup-updates branch from 74728c8 to 0afd1c3 Compare May 30, 2023 10:24
@alistairjcbrown alistairjcbrown force-pushed the daniel/macos-incontext-signup-updates branch from 0afd1c3 to 0c818a7 Compare May 30, 2023 10:29
@alistairjcbrown alistairjcbrown force-pushed the daniel/macos-incontext-signup-updates branch 2 times, most recently from 33e0123 to 6250bda Compare May 31, 2023 17:22
@alistairjcbrown alistairjcbrown force-pushed the daniel/macos-incontext-signup-updates branch from 6250bda to 9475ad4 Compare June 2, 2023 11:23
@alistairjcbrown
Copy link
Member Author

Autofill update commit cherry-picked from #377

@amddg44 amddg44 self-assigned this Jun 2, 2023
@amddg44 amddg44 merged commit 2e90e6c into main Jun 9, 2023
@amddg44 amddg44 deleted the daniel/macos-incontext-signup-updates branch June 9, 2023 13:26
@amddg44 amddg44 restored the daniel/macos-incontext-signup-updates branch June 9, 2023 13:49
samsymons added a commit that referenced this pull request Jun 19, 2023
* main: (45 commits)
  SecureVault Keychain updates (#382)
  Reattach orphaned bookmarks to root folder when reordering (#383)
  Allow for asynchronous sync initialization (#381)
  Drop regex support in param stripping (#367)
  mac promo exp2 - add support for share link message (#375)
  Revert accidentally pushed change
  Remove test that was dependant on the keychain
  Add item to keychain in the background (#380)
  macOS in-context signup updates (#359)
  Fixes the format in a log call (#379)
  Add EventMapping to DDGSync dependencies and call it from handleUnauthenticated (#376)
  Autofill hash migration updates + Bitwarden hotfix (#372)
  Autofill password generation support for iOS (#361)
  Update autofill to 7.1.0 (#370)
  Sync Engine with support for syncing bookmarks (#355)
  Centralised Messaging for User Scripts (#299)
  Update TLD accounts query to be more specific (#368)
  Bump Tests/BrowserServicesKitTests/Resources/privacy-reference-tests (#366)
  Adds a different hash Account per each macOS build kind (#363)
  adjust os_log parameter name using disfavoredOverload (#349)
  ...
@afterxleep afterxleep deleted the daniel/macos-incontext-signup-updates branch July 17, 2023 12:41
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.

4 participants