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

mac promo exp2 - add support for share link message #375

Merged
merged 6 commits into from
Jun 15, 2023

Conversation

brindy
Copy link
Contributor

@brindy brindy commented May 31, 2023

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/72649045549333/1204632216628999/f
iOS PR: duckduckgo/iOS#1755
macOS PR:
What kind of version bump will this require?: Major/Minor/Patch

Optional:

Tech Design URL:
CC:

Description:

Steps to test this PR:

  1. See iOS PR: Mac Promo Experiment 2 iOS#1755
  2. Note that the message request in iOS is currently pointing to https://raw.githubusercontent.com/duckduckgo/remote-messaging-config/brindy/mac-promo-exp2/live/ios-config/ios-config.json - ensure that the JSON can be parsed
  3. Smoke test parsing against known good JSON

OS Testing:

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

Internal references:

Software Engineering Expectations
Technical Design Template

@brindy brindy self-assigned this May 31, 2023
@brindy brindy mentioned this pull request May 31, 2023
13 tasks
@brindy brindy requested a review from amddg44 May 31, 2023 16:05
@brindy brindy assigned amddg44 and unassigned brindy May 31, 2023
@brindy brindy marked this pull request as ready for review May 31, 2023 16:05
Copy link
Contributor

@amddg44 amddg44 left a comment

Choose a reason for hiding this comment

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

LGTM!

@brindy brindy assigned brindy and unassigned amddg44 Jun 15, 2023
@brindy brindy merged commit c6bc446 into main Jun 15, 2023
@brindy brindy deleted the brindy/mac-promo-exp2 branch June 15, 2023 12:11
brindy added a commit to duckduckgo/iOS that referenced this pull request Jun 15, 2023
Task/Issue URL: https://app.asana.com/0/72649045549333/1204632216628999/f
Tech Design URL:
CC:

Description:

Adds support for Mac Promo Experiment 2 which shows a full screen / page sheet view and tests against the remote message framework.

Minor changes were made to BSK (see duckduckgo/BrowserServicesKit#375 )
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)
  ...
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