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

Fixes the format in a log call #379

Merged
merged 1 commit into from
Jun 9, 2023

Conversation

diegoreymendez
Copy link
Contributor

@diegoreymendez diegoreymendez commented Jun 8, 2023

Task/Issue URL: https://app.asana.com/0/0/1204785816528431/f
iOS PR: duckduckgo/iOS#1760
macOS PR: duckduckgo/macos-browser#1249
What kind of version bump will this require?: Patch
CC: @ayoy

Description

Fixes a string formatting issue that crashes the app in debug builds, when Bookmarks logging is ON.

Testing

Please refer to the iOS and macOS PRs for testing steps.

OS Testing

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

Internal references:

Software Engineering Expectations
Technical Design Template

@diegoreymendez diegoreymendez removed the request for review from ayoy June 8, 2023 16:01
@diegoreymendez diegoreymendez merged commit a1c93d2 into main Jun 9, 2023
@diegoreymendez diegoreymendez deleted the diego/fix-debug-crash-on-bookmarks-log-call branch June 9, 2023 08:08
diegoreymendez added a commit to duckduckgo/iOS that referenced this pull request Jun 9, 2023
Task/Issue URL: https://app.asana.com/0/0/1204785816528431/f
BSK: duckduckgo/BrowserServicesKit#379
macOS: duckduckgo/macos-browser#1249

Description

Fixes a string formatting issue that crashes the app in debug builds, when Bookmarks logging is ON.
diegoreymendez added a commit to duckduckgo/macos-browser that referenced this pull request Jun 9, 2023
Task/Issue URL:
https://app.asana.com/0/1203108348835387/1204785816528431/f
BSK: duckduckgo/BrowserServicesKit#379
iOS: duckduckgo/iOS#1760

## Description:

Fixes a string formatting issue that crashes the app in debug builds,
when Bookmarks logging is ON.
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