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

Simplify connecting Sync accounts when 2 exist #3867

Merged
merged 25 commits into from
Jan 27, 2025

Conversation

graeme
Copy link
Contributor

@graeme graeme commented Jan 24, 2025

Task/Issue URL: https://app.asana.com/0/72649045549333/1208944782348823/f

Description:

Reopened #3747

Around 600 daily Sync users try to connect a Synced device with another Sync Backup, which is not possible, and we show them an error indicating they should stop syncing first and try again.

Simplify the steps by:

Automatically Switch Sync for users with no other devices connected. (This is the most common scenario) -
Ask, and allow them to continue switching sync for users with more devices connected.
Steps to test this PR:
https://app.asana.com/0/0/1208993297756834/f

also confirm that pixels are sent:
https://app.asana.com/0/1142021229838617/1208928557675008/f

Definition of Done (Internal Only):

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than '

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPhone 14 Pro
  • iPad

OS Testing:

  • iOS 15
  • iOS 16
  • iOS 17

Theme Testing:

  • Light theme
  • Dark theme

Internal references:

Software Engineering Expectations
Technical Design Template

Copy link

github-actions bot commented Jan 24, 2025

Messages
📖

You seem to be updating localized strings. Make sure that you request translations and include translated strings before you ship your change. See Localization Guidelines for more information.

Generated by 🚫 dangerJS against ec9f48b

@graeme graeme changed the title Simplify connecting ync accounts when 2 exist Simplify connecting Sync accounts when 2 exist Jan 24, 2025
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! Just heads up on the swiftlint cyclomatic_complexity warning 🙂

graeme added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Jan 27, 2025
Please review the release process for BrowserServicesKit
[here](https://app.asana.com/0/1200194497630846/1200837094583426).

**Required**:

Task/Issue URL:
https://app.asana.com/0/1142021229838617/1209000370243820/f
iOS PR: duckduckgo/iOS#3867
macOS PR: duckduckgo/macos-browser#3779
What kind of version bump will this require?: Major

**Description**:

Just a feature flag. Please see platform PRs for testing.

<!--
Before submitting a PR, please ensure you have tested the combinations
you expect the reviewer to test, then delete configurations you *know*
do not need explicit testing.

Using a simulator where a physical device is unavailable is acceptable.
-->

**OS Testing**:

* [ ] iOS 14
* [ ] iOS 15
* [ ] iOS 16
* [ ] macOS 10.15
* [ ] macOS 11
* [ ] macOS 12

---
###### Internal references:
[Software Engineering
Expectations](https://app.asana.com/0/59792373528535/199064865822552)
[Technical Design
Template](https://app.asana.com/0/59792373528535/184709971311943)
@graeme graeme merged commit 3c06593 into main Jan 27, 2025
13 checks passed
@graeme graeme deleted the graeme/simplify-connecting-sync-accounts-when-2 branch January 27, 2025 13:51
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