Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Conversation

brindy
Copy link
Contributor

@brindy brindy commented Sep 19, 2023

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

Description:

Steps to test this PR:

  1. Build and run the app.
  2. Add and delete a bookmark.

Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

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 😄

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
@brindy brindy merged commit 41ab951 into develop Sep 21, 2023
@brindy brindy deleted the brindy/update-bsk-for-delete-bookmark branch September 21, 2023 11:03
samsymons added a commit that referenced this pull request Sep 24, 2023
…ove-hardcoded-message

# By Fernando Bunn (4) and others
# Via Sam Symons (2) and GitHub (1)
* develop: (23 commits)
  Add run attempt to WORKFLOW_URL for reporting failed tests (#1671)
  Fix share in News (#1668)
  Set version to 1.57.1.
  Update BSK version for previous cherry-pick.
  The connection tester no longer disables on-demand (#1631)
  Home Button Implementation (#1649)
  Fix issue not setting maintenance date (#1663)
  Update DebugUI (#1659)
  Fix DBP on close notification (#1652)
  Ask for full disk access (NSAppDataUsageDescription) (#1639)
  Log user out of Sync when SyncOperation receives a HTTP 401 (#1666)
  fix first responder, improve logging, minor refactoring (#1657)
  Fix unit tests (#1662)
  Add child broker json files (#1660)
  Update HomePageContinueSetUpModel to not save a hardcoded Privacy Config reference (#1637)
  Update OpenSSL to 3.1.2000. (#1636)
  Onboarding flow fix for macOS Sonoma (#1655)
  Update profile address copy (#1658)
  Add WireGuard NetP Error Pixels (#1641)
  update bsk for bookmarks view model changes (#1640)
  ...

# Conflicts:
#	DuckDuckGo/HomePage/Model/HomePageContinueSetUpModel.swift
samsymons added a commit that referenced this pull request Sep 24, 2023
…message' into sam/add-netp-messaging-support-part-2-add-remote-fetching

# By Fernando Bunn (4) and others
# Via Sam Symons (3) and GitHub (1)
* sam/add-netp-messaging-support-part-1-remove-hardcoded-message: (23 commits)
  Add run attempt to WORKFLOW_URL for reporting failed tests (#1671)
  Fix share in News (#1668)
  Set version to 1.57.1.
  Update BSK version for previous cherry-pick.
  The connection tester no longer disables on-demand (#1631)
  Home Button Implementation (#1649)
  Fix issue not setting maintenance date (#1663)
  Update DebugUI (#1659)
  Fix DBP on close notification (#1652)
  Ask for full disk access (NSAppDataUsageDescription) (#1639)
  Log user out of Sync when SyncOperation receives a HTTP 401 (#1666)
  fix first responder, improve logging, minor refactoring (#1657)
  Fix unit tests (#1662)
  Add child broker json files (#1660)
  Update HomePageContinueSetUpModel to not save a hardcoded Privacy Config reference (#1637)
  Update OpenSSL to 3.1.2000. (#1636)
  Onboarding flow fix for macOS Sonoma (#1655)
  Update profile address copy (#1658)
  Add WireGuard NetP Error Pixels (#1641)
  update bsk for bookmarks view model changes (#1640)
  ...

# Conflicts:
#	DuckDuckGo/HomePage/Model/HomePageContinueSetUpModel.swift
#	DuckDuckGo/Statistics/PixelEvent.swift
#	DuckDuckGo/Statistics/PixelParameters.swift
#	UnitTests/HomePage/ContinueSetUpModelTests.swift
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants