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 May 17, 2023

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/0/1204469803916196/f
iOS PR: duckduckgo/iOS#1727
macOS PR: duckduckgo/macos-browser#1205
What kind of version bump will this require?: Minor

Optional:

Tech Design URL:
CC:

Description:
Returns devices returned by the login URL. Update iOS and macOS to show the new devices that are connected.

Steps to test this PR:

  1. On iOS turn off Sync
  2. On macOS turn on Sync
  3. Use iOS to connect the Sync on your macOS
  4. Both devices should show the other device when they connect
  5. Confirm this works the opposite way around too

OS Testing:

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

Internal references:

Software Engineering Expectations
Technical Design Template

Copy link
Contributor

@ayoy ayoy 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 assigned brindy and unassigned ayoy May 17, 2023
@brindy brindy merged commit b19b5e4 into main May 17, 2023
@brindy brindy deleted the brindy/sync-device-connected-names branch May 17, 2023 18:37
brindy added a commit to duckduckgo/macos-browser that referenced this pull request May 17, 2023
Task/Issue URL: https://app.asana.com/0/0/1204469803916196/f
Tech Design URL:
CC:

**Description**:
Shows real connected device name.

**Steps to test this PR**:
1. See duckduckgo/BrowserServicesKit#357

<!--
Tagging instructions
If this PR isn't ready to be merged for whatever reason it should be
marked with the `DO NOT MERGE` label (particularly if it's a draft)
If it's pending Product Review/PFR, please add the `Pending Product
Review` label.

If at any point it isn't actively being worked on/ready for
review/otherwise moving forward (besides the above PR/PFR exception)
strongly consider closing it (or not opening it in the first place). If
you decide not to close it, make sure it's labelled to make it clear the
PRs state and comment with more information.
-->

---
###### Internal references:
[Pull Request Review
Checklist](https://app.asana.com/0/1202500774821704/1203764234894239/f)
[Software Engineering
Expectations](https://app.asana.com/0/59792373528535/199064865822552)
[Technical Design
Template](https://app.asana.com/0/59792373528535/184709971311943)
[Pull Request
Documentation](https://app.asana.com/0/1202500774821704/1204012835277482/f)
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