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

Conversation

@ayoy
Copy link
Collaborator

@ayoy ayoy commented Aug 23, 2023

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

Description:
Only clean database when sync is inactive at app startup.

Steps to test this PR:

  1. Run the app from Xcode while Sync is not active
  2. Verify that bookmarks and credentials database cleanup was attempted ("No bookmarks pending deletion" and "No syncable credentials metadata pending deletion" messages appear in the log).
  3. Enable Sync, verify that bookmarks and credentials cleanup was attempted, then quit the app.
  4. Relaunch the app, verify that bookmarks and credentials cleanup was not attempted.
  5. Disable Sync, verify that bookmarks and credentials cleanup was attempted.

Internal references:

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

@ayoy ayoy force-pushed the dominik/fix-sync-metadata-cleanup branch from de0fa3a to 2c46839 Compare August 23, 2023 11:03
@ayoy ayoy requested a review from bwaresiak August 23, 2023 11:26
@ayoy ayoy assigned bwaresiak and unassigned ayoy Aug 23, 2023
@ayoy ayoy marked this pull request as ready for review August 23, 2023 11:27
Copy link
Collaborator

@bwaresiak bwaresiak left a comment

Choose a reason for hiding this comment

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

LGTM!

@ayoy ayoy merged commit c9a1a6a into develop Aug 23, 2023
@ayoy ayoy deleted the dominik/fix-sync-metadata-cleanup branch August 23, 2023 16:21
samsymons added a commit that referenced this pull request Aug 23, 2023
# By Diego Rey Mendez (7) and others
# Via Sam Symons (2) and others
* develop: (26 commits)
  Improve Sync-related database cleaning logic (#1529)
  Update onboarding-related error states (#1504)
  Prevents launching our menu agent without an auth code. (#1516)
  Autofill UI letter icons (#1535)
  Cleans up some code (#1517)
  Revert "Autofill Letter Icons" (#1534)
  Adds remote pre-commit installer, which includes automatic fix for linter (#1369)
  Autofill Letter Icons (#1475)
  change context menu for mailto links (#1513)
  Updates the version to 1.53.1
  Updated the embedded files for 1.53.1
  Update the phased rollout tester to avoid caching the config (#1520)
  Require Duck Player scheme URL to be passed from YouTube Overlay User Script (#1519)
  Add pixels related to Duck Player usage (#1515)
  only allow error reloads on http(s) urls (#1523)
  Standardize TDS Loading Error handling (#1524)
  Move pixel sender logic into the main view controller (#1528)
  Update the phased rollout tester to avoid caching the config (#1520)
  Set version to 1.52.3.
  Move pixel sender logic into the main view controller (#1528)
  ...

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	DuckDuckGo/AppDelegate/AppDelegate.swift
#	DuckDuckGo/Common/Localizables/UserText.swift
#	DuckDuckGo/Common/Utilities/UserDefaultsWrapper.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