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

VPN location fallback #953

Merged
merged 4 commits into from
Aug 29, 2024
Merged

VPN location fallback #953

merged 4 commits into from
Aug 29, 2024

Conversation

quanganhdo
Copy link
Member

@quanganhdo quanganhdo commented Aug 21, 2024

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/1205591970852438/1206773442486707/f
iOS PR: duckduckgo/iOS#3264
macOS PR: duckduckgo/macos-browser#3131
What kind of version bump will this require?: Major/Minor/Patch

Optional:

Tech Design URL:
CC:

Description:

This PR updates @graeme's geolocation PRs up to date with changes from BSK and address some prior review feedback.

Steps to test this PR:

Check platform-specific PRs.

OS Testing:

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

Internal references:

Software Engineering Expectations
Technical Design Template

@quanganhdo quanganhdo self-assigned this Aug 21, 2024
@quanganhdo quanganhdo marked this pull request as ready for review August 21, 2024 17:29
@quanganhdo quanganhdo requested a review from samsymons August 21, 2024 17:30
# Conflicts:
#	Sources/NetworkProtection/NetworkProtectionDeviceManager.swift
#	Sources/NetworkProtection/PacketTunnelProvider.swift
@quanganhdo quanganhdo merged commit 1ee0979 into main Aug 29, 2024
7 checks passed
@quanganhdo quanganhdo deleted the anh/netp/geolocation-issues branch August 29, 2024 14:16
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