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 for deeplink crash when preparing webview preview #2116

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

amddg44
Copy link
Contributor

@amddg44 amddg44 commented Oct 25, 2023

Task/Issue URL: https://app.asana.com/0/414709148257752/1205793869765618/f
Tech Design URL:
CC:

Description:
When launching the app from deeplink the webView size can be 0,0 causing UIGraphicsBeginImageContextWithOptions to crash

Steps to test this PR:

  1. Add this link somewhere clickable so that tapping it will launch the DDG app e.g. mail, reminders
  2. Force quit the app if it's already running
  3. Tap the link and confirm the app launches without issue, the webview loads correctly and the tab switcher is correctly populated
  4. Smoke test adding tabs, including in background + confirm no regression of fix ui corruption after adding pass to wallet #2081

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than '

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

…thOptions crashing when the webview size is 0,0
@amddg44 amddg44 requested a review from brindy October 25, 2023 14:15
Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for fixing that.

@amddg44 amddg44 merged commit ce8ce75 into develop Oct 31, 2023
9 checks passed
@amddg44 amddg44 deleted the anya/fix-deeplink-webview-preview-crash branch October 31, 2023 07:27
samsymons added a commit that referenced this pull request Nov 5, 2023
* develop:
  Alert user about abnormal app conditions (#2110)
  Add e2e test for email protection deep-links (#2123)
  Update to config v4 (#2114)
  Release 7.95.0 (#2122)
  Add toggling of NetP Notifications to iOS (#2112)
  Fix for deeplink crash when preparing webview preview (#2116)
  Integrates the latest DBP changes in BSK (#2108)
  Switch to next phase of new experiment for UA (#2118)
samsymons added a commit that referenced this pull request Nov 8, 2023
* develop:
  re-enable keyboard shortcuts (#2132)
  fix favorite launch with keyboard bug (#2131)
  Fix syncing empty favorites folders (#2121)
  kill variant when receiving atb update (#2130)
  Adding 'protectionsState' to breakage form submission (#2120)
  Release 7.96.0 (#2128)
  Update remote messaging production url (#2124)
  Alert user about abnormal app conditions (#2110)
  Add e2e test for email protection deep-links (#2123)
  Update to config v4 (#2114)
  Release 7.95.0 (#2122)
  Add toggling of NetP Notifications to iOS (#2112)
  Fix for deeplink crash when preparing webview preview (#2116)
  Integrates the latest DBP changes in BSK (#2108)
  Switch to next phase of new experiment for UA (#2118)
samsymons added a commit that referenced this pull request Nov 9, 2023
* develop:
  Release 7.95.0 (#2122)
  Add toggling of NetP Notifications to iOS (#2112)
  Fix for deeplink crash when preparing webview preview (#2116)
  Integrates the latest DBP changes in BSK (#2108)
  Switch to next phase of new experiment for UA (#2118)
  Update BSK with autofill 9.0.0 (#2103)
  Add an “Open VPN Settings” quick action (#2089)
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