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

prevent devices from being fetch when sync not visible #1214

Merged
merged 1 commit into from
May 24, 2023

Conversation

brindy
Copy link
Contributor

@brindy brindy commented May 23, 2023

Task/Issue URL: https://app.asana.com/0/1177771139624306/1204659411978394/f
Tech Design URL:
CC:

Description:

Prevent calls to fetch devices when not viewing sync preferences.

Steps to test this PR:

  1. Monitor network traffic or add print statement in refreshDevices call
  2. Open sync preferences
  3. Enable sync if needed
  4. Observe call to fetch devices
  5. Move to another section in the preferences
  6. Observe no calls to fetch devices
  7. Return to sync preference
  8. Observe calls to fetch devices
  9. Close tab
  10. Observe no calls to fetch devices

Internal references:

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

@brindy brindy requested a review from ayoy May 23, 2023 21:03
Copy link
Collaborator

@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.

Works great 👏

@ayoy ayoy assigned brindy and unassigned ayoy May 24, 2023
@brindy brindy merged commit 9aaa430 into develop May 24, 2023
@brindy brindy deleted the brindy/sync-stop-device-fetch branch May 24, 2023 09:59
samsymons added a commit that referenced this pull request May 25, 2023
# By Juan Pereira (2) and others
# Via GitHub (1) and Juan Pereira (1)
* upstream_develop:
  add newTab pixel, fire pixels only for new users first time (#1219)
  Update tds endpoint (#1218)
  prevent devices from being fetch when sync not visible (#1214)
  License and contributor guidelines for open-sourcing (#1189)
  Add a unique Bitwarden decryption failure pixel (#1197)
  Bump version to 1.41.0 (27)
  Update embedded files

# Conflicts:
#	DuckDuckGo/Statistics/PixelEvent.swift
#	DuckDuckGo/Statistics/PixelParameters.swift
samsymons added a commit that referenced this pull request May 27, 2023
* release/1.42.0: (26 commits)
  Bump version to 1.42.0 (28)
  Update embedded files
  Network Protection (#1211)
  Update BSK with autofill 7.1.0 (#1225)
  Autofill password generation support for iOS (#1212)
  Sync Engine with support for syncing bookmarks (#1203)
  Bump BSK for messaging updates (#1213)
  add newTab pixel, fire pixels only for new users first time (#1219)
  Update tds endpoint (#1218)
  prevent devices from being fetch when sync not visible (#1214)
  License and contributor guidelines for open-sourcing (#1189)
  Add a unique Bitwarden decryption failure pixel (#1197)
  Bump version to 1.41.0 (27)
  Update embedded files
  Check that the source and target frame security origins are equal (#1207)
  Add activation points pixels (#1206)
  Add Login export links to settings and Autofill 3 dot menu (#1183)
  sync device connected names (#1205)
  Autofill/subdomain matching (#1122)
  add-windows-browser-download-link-privacy-config (#1198)
  ...
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