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

Don't crash on assert in alpha builds #2667

Merged
merged 1 commit into from
Apr 3, 2024
Merged

Conversation

graeme
Copy link
Contributor

@graeme graeme commented Apr 3, 2024

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

Description:
As we saw in Crashes when opening from external links with auto-clear+expiry alpha builds can crash on asserts

Steps to test this PR:
Can only be tested after merge. Use https://app.asana.com/0/414709148257752/1206731143449260 as a test case that was previously crashing. But if you want to test the added config, you could.

  1. Change the added Configuration-Alpha.xcconfig OTHER_SWIFT_FLAGS line to be Alpha Debug.
  2. Follow the steps in the above task.

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

@graeme graeme requested a review from brindy April 3, 2024 14:13
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

@samsymons
Copy link
Contributor

Going to merge this so that we can get it out in tonight's alpha and validate it. Thanks for this!

@samsymons samsymons merged commit 760b58a into main Apr 3, 2024
18 checks passed
@samsymons samsymons deleted the graeme/alpha-no-assert-crash branch April 3, 2024 21:24
afterxleep added a commit that referenced this pull request Apr 4, 2024
afterxleep added a commit that referenced this pull request Apr 4, 2024
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