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

iOS VPN Settings part 1 #2109

Merged
merged 12 commits into from
Oct 19, 2023
Merged

iOS VPN Settings part 1 #2109

merged 12 commits into from
Oct 19, 2023

Conversation

graeme
Copy link
Contributor

@graeme graeme commented Oct 19, 2023

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

Description:

Adds the new VPN Settings items + the VPN Settings screen. The next PR (part 2) will include the implementation for the VPN Notifications screen.

Steps to test this PR:

  1. Make sure you are an internal user (Settings -> Debug -> Internal User). Make sure you close the Settings before proceeding.
  2. Navigate to the Network Protection Status screen (Settings -> Network Protection).
  3. Observe the section title Manage with the two new items: VPN Settings & VPN Notifications
  4. Select VPN Settings
  5. Observe the two sections each with a disabled toggle item for: Always On & Secure DNS
  6. Generally check that everything matches the designs

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

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

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16

Theme Testing:

  • Light theme
  • Dark theme

Internal references:

Software Engineering Expectations
Technical Design Template

)
toggleSection(
text: UserText.netPSecureDNSSettingTitle,
footerText: UserText.netPSecureDNSSettingFooter
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The designs have a “Learn more” link, but we’re not including that just yet. https://app.asana.com/0/0/1204630981406975/1205756069414403/f

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hah, good note, I was about to report it missing! :)

@graeme graeme requested a review from diegoreymendez October 19, 2023 12:51
Copy link
Contributor

@diegoreymendez diegoreymendez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! Approved.

@graeme graeme merged commit 5c5ee6d into develop Oct 19, 2023
9 checks passed
@graeme graeme deleted the graeme/ios-vpn-settings branch October 19, 2023 13:13
samsymons added a commit that referenced this pull request Oct 23, 2023
* develop:
  bump BSK (changed dashboard API) (#2111)
  Integrate BSK changes (#2101)
  bump bsk for duckplayer fixes on BigSur (#2106)
  iOS VPN Settings part 1 (#2109)
  Release 7.94.0 (#2107)
  Append build number to metricKit crash version (#2100)
  Disable NetP errors temporarily (#2102)
  NetP: Fix server location formatting (#2099)
  Release 7.93.0 (#2096)
samsymons added a commit that referenced this pull request Oct 27, 2023
* develop: (28 commits)
  Update BSK with autofill 9.0.0 (#2103)
  Add an “Open VPN Settings” quick action (#2089)
  bump BSK (changed dashboard API) (#2111)
  Integrate BSK changes (#2101)
  bump bsk for duckplayer fixes on BigSur (#2106)
  iOS VPN Settings part 1 (#2109)
  Release 7.94.0 (#2107)
  Append build number to metricKit crash version (#2100)
  Disable NetP errors temporarily (#2102)
  NetP: Fix server location formatting (#2099)
  address bar position (#2072)
  Fix privacy tests (#2097)
  Add note about Danger matching rules (#2087)
  Release 7.93.0 (#2096)
  Release 7.93.0 (#2096)
  Update tabs tests (#2088)
  Ignore form factor specific favorites if they're received in Sync response (#2083)
  Update BSK with autofill 8.4.2 (#2084)
  Bump C-S-S to 4.39.0 (#2085)
  BSK hotfix 80.3.0-1 merged to the main branch (#2079)
  ...
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