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

Sabrina/sync setup update #2198

Merged
merged 29 commits into from
Dec 1, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
8c9a3ee
implement new flow
SabrinaTardio Nov 24, 2023
9e554ab
clean up Sync set up and sync enabled page
SabrinaTardio Nov 24, 2023
6dcd4bc
clean up connect with server sheet
SabrinaTardio Nov 24, 2023
5a08ba3
clean up preparing to sync sheetgit status
SabrinaTardio Nov 24, 2023
d27f862
clean up preparing to sync sheet
SabrinaTardio Nov 24, 2023
d4bf7b1
clean up save recovery key view
SabrinaTardio Nov 27, 2023
fdac074
clean up device connected view
SabrinaTardio Nov 27, 2023
56cc912
clean up recover synced data view
SabrinaTardio Nov 27, 2023
4760be6
recover code camera view clean up
SabrinaTardio Nov 27, 2023
5d45bc6
clean manually enter code view
SabrinaTardio Nov 27, 2023
c041451
clean up scan or view code
SabrinaTardio Nov 27, 2023
a2785c1
fix connect two unsynced devices path
SabrinaTardio Nov 27, 2023
92c9b3e
remove unwanted changes
SabrinaTardio Nov 27, 2023
acfa3b3
clean up some more
SabrinaTardio Nov 27, 2023
09738ac
fix tests
SabrinaTardio Nov 27, 2023
a5b1a1e
handle better a negative scenario
SabrinaTardio Nov 28, 2023
fce6b0d
fix lint issue
SabrinaTardio Nov 28, 2023
9d47a8f
address comments
SabrinaTardio Nov 30, 2023
28fe773
fix headers
SabrinaTardio Nov 30, 2023
1572ac5
Merge branch 'develop' into sabrina/sync-setup-update
SabrinaTardio Nov 30, 2023
61239b2
fix copy
SabrinaTardio Nov 30, 2023
904587b
fix alignment
SabrinaTardio Nov 30, 2023
83d0e00
show invalid code error
SabrinaTardio Nov 30, 2023
ed135bb
fix bug when QR code is not shown
SabrinaTardio Nov 30, 2023
35c46e8
remove unwanted change
SabrinaTardio Nov 30, 2023
5e12d37
kind of fix racing condition
SabrinaTardio Nov 30, 2023
519cb4f
fix pipeline
SabrinaTardio Nov 30, 2023
16c1ec7
fix pipeline hopefully
SabrinaTardio Nov 30, 2023
46f3b2e
update file name
SabrinaTardio Dec 1, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion DuckDuckGo/SettingsViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ class SettingsViewController: UITableViewController {
}

private var shouldShowSyncCell: Bool {
return featureFlagger.isFeatureOn(.sync)
return true
SabrinaTardio marked this conversation as resolved.
Show resolved Hide resolved
}

private var shouldShowTextSizeCell: Bool {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,8 @@ struct UserText {
static let nextButton = NSLocalizedString("next.button", value: "Next", comment: "Standard Buttons - Next Button")
static let backButton = NSLocalizedString("back.button", value: "Back", comment: "Standard Buttons - Back Button")
static let pasteButton = NSLocalizedString("paste.button", value: "Paste", comment: "Standard Buttons - Paste Button")

static let notNowButton = NSLocalizedString("not.now.button", value: "Not Now", comment: "Standard Buttons - Not Now Button")

// Fetch favicons
static let fetchFaviconsOnboardingTitle = NSLocalizedString("fetch.favicons.onboarding.title", value: "Download Missing Icons?", comment: "Fetch Favicons Onboarding - Title")
static let fetchFaviconsOnboardingMessage = NSLocalizedString("fetch.favicons.onboarding.message", value: "Do you want this device to automatically download icons for any new bookmarks synced from your other devices? This will expose the download to your network any time a bookmark is synced.", comment: "Fetch Favicons Onboarding - Message")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -273,18 +273,20 @@ extension SyncSettingsView {
Toggle(isOn: $model.isFaviconsFetchingEnabled) {
VStack(alignment: .leading, spacing: 5) {
Text(UserText.fetchFaviconsOptionTitle)
.daxBodyRegular()
.foregroundColor(.primary)
Text(UserText.fetchFaviconsOptionCaption)
.daxBodyRegular()
.daxFootnoteRegular()
.foregroundColor(.secondary)
}
}
Toggle(isOn: $model.isUnifiedFavoritesEnabled) {
VStack(alignment: .leading, spacing: 5) {
Text(UserText.unifiedFavoritesTitle)
.daxBodyRegular()
.foregroundColor(.primary)
Text(UserText.unifiedFavoritesInstruction)
.daxBodyRegular()
.daxFootnoteRegular()
.foregroundColor(.secondary)
}
}
Expand Down
Loading