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

Fix widgets init due to crash of availability check inside builder on iOS 16 #2931

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

miasma13
Copy link
Contributor

@miasma13 miasma13 commented Jun 7, 2024

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

Description:
It turns out that if #available(iOSApplicationExtension 17.0, *) check within the body of the builder is causing a crash on iOS 16. The change was introduced in most recent version by making the widget available in release target.

Steps to test this PR:

  • Build WidgetsExtension target against iOS 16 simulator - it should not crash
  • Run the app on iOS 16 and try adding home screen widgets from DDG app - the VPN one should not be available
  • Run the app on iOS 17 and try adding home screen widgets from DDG app - the VPN one should be available

Device Testing:

  • iPhone
  • iPad

OS Testing:

  • iOS 17
  • iOS 16

Internal references:

Software Engineering Expectations
Technical Design Template

@miasma13 miasma13 changed the title Rework how the widgets are built due to crash of availability check o… Fix widgets init due to crash of availability check inside builder on iOS 16 Jun 7, 2024
@ayoy ayoy self-requested a review June 7, 2024 10:17
@amddg44 amddg44 self-requested a review June 7, 2024 10:19
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! Tested with iOS 16 and iOS 15 and I'm not getting crashes 👏

Copy link
Contributor

@amddg44 amddg44 left a comment

Choose a reason for hiding this comment

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

LGTM also! 👏

@miasma13 miasma13 merged commit 6086d15 into hotfix/7.122.1 Jun 7, 2024
34 checks passed
@miasma13 miasma13 deleted the michal/fix-vpn-widget-crash-on-ios16 branch June 7, 2024 10:31
samsymons added a commit that referenced this pull request Jun 7, 2024
* main: (34 commits)
  Release 7.122.1-0 (#2932)
  Fix widgets init due to crash of availability check inside builder on iOS 16 (#2931)
  Show subscription settings section either when subscription purchase is possible or user is authenticated (#2922)
  Release 7.123.0-1 (#2928)
  Do not refresh UI only if AutoClear was triggered (#2924)
  Do not refresh UI only if AutoClear was triggered (#2924)
  Clean up subscription state caching in settings logic (#2925)
  Force Omni Bar cancellation when submitting a query (#2923)
  Hide 'Set your address bar position' for iPad (#2913)
  Select correct access group for SubscriptionTokenKeychainStorage (#2918)
  Remove VPN waitlist feature flags and access controller (#2919)
  Remove usage of token store from iOS VPN (#2892)
  Update autoconsent to v10.9.0 (#2908)
  Load child Privacy Pro views lazily (#2917)
  Surface specific XPC & login item errors on macOS (#2872)
  Removing the temporary password manager survey code (#2910)
  Release 7.123.0-0 (#2915)
  Use better wording for history autocomplete description (#2914)
  Additional Privacy Pro RMF changes (#2911)
  Bump BSK - No iOS Changes, Enables macOS DBP Changes (#2877)
  ...
samsymons added a commit that referenced this pull request Jun 9, 2024
# By Michal Smaga (6) and others
# Via Michal Smaga (3) and GitHub (1)
* main:
  DBP: Bump C-S-S version to 5.19.0 (#2927)
  Release 7.123.0-2 (#2933)
  Fix widgets init due to crash of availability check inside builder on iOS 16 (#2931)
  Release 7.122.1-0 (#2932)
  Fix widgets init due to crash of availability check inside builder on iOS 16 (#2931)
  Show subscription settings section either when subscription purchase is possible or user is authenticated (#2922)
  Release 7.123.0-1 (#2928)
  Do not refresh UI only if AutoClear was triggered (#2924)
  Clean up subscription state caching in settings logic (#2925)
  Force Omni Bar cancellation when submitting a query (#2923)
  Remove Privacy Pro line from release notes (#2902)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
samsymons added a commit that referenced this pull request Jun 10, 2024
* main:
  Add Privacy Pro survey parameters (#2920)
  DBP: Bump C-S-S version to 5.19.0 (#2927)
  Release 7.123.0-2 (#2933)
  Fix widgets init due to crash of availability check inside builder on iOS 16 (#2931)
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.

3 participants