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

Remove timezone offset from the VPN server object #2580

Merged
merged 3 commits into from
Apr 12, 2024
Merged

Conversation

samsymons
Copy link
Collaborator

@samsymons samsymons commented Apr 9, 2024

Task/Issue URL: https://app.asana.com/0/414235014887631/1207032029127388/f
Tech Design URL:
CC:

Description:

This PR removes support for tzOffset, which is no longer used.

Steps to test this PR:

  1. Test that geoswitching works
  2. Test the the server list in the Debug menu still returns servers as expected

Internal references:

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

samsymons added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Apr 12, 2024
Required:

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

Description:

This PR removes tzOffset from the server object. It wasn't being used and is in the process of being removed.
# By Dax the Duck (10) and others
# Via GitHub (6) and others
* main: (40 commits)
  Bump version to 1.83.0 (160)
  macOS VPN: Ask users to reboot if system extension was not uninstalled (#2603)
  macOS VPN: Ask users to reboot if system extension was not uninstalled (#2603)
  Fix popover not displayed reliably when VPN shortcut is unpinned (#2606)
  Automatically mark / close stale PRs (#2596)
  Update copy for DBP open button (#2586)
  Bump version to 1.83.0 (159)
  [Release PR] Fix lottie high Windowserver load (#2598)
  Bump version to 1.83.0 (158)
  BSK release 133.1.0 (#2597)
  Fix VPN bug: Nearest city breaks register requests (#2589)
  Fix lottie high Windowserver load (#2595)
  drop Downloads storyboard (#2556)
  Disable directory download (#2585)
  Add supported document types (#2581)
  Allow choosing downloads location in App Store builds (#2532)
  Fix Open Downloads not working (#2576)
  Update Privacy Dashboard URL on navigation commit (#2583)
  Percent-decode download filenames (#2584)
  Bump version to 1.83.0 (157)
  ...

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
@samsymons samsymons merged commit 5da74bf into main Apr 12, 2024
18 checks passed
@samsymons samsymons deleted the sam/remove-tzoffset branch April 12, 2024 04:23
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