Skip to content

Aztec: shouldInteractWithURL will return always false to avoid crashes#12798

Merged
etoledom merged 1 commit intodevelopfrom
issue/12730-interact-with-url-crash
Nov 4, 2019
Merged

Aztec: shouldInteractWithURL will return always false to avoid crashes#12798
etoledom merged 1 commit intodevelopfrom
issue/12730-interact-with-url-crash

Conversation

@etoledom
Copy link
Copy Markdown
Contributor

@etoledom etoledom commented Oct 29, 2019

Fixes #12730

Following up from #12772.
I agree with @koke on #12772 (comment).

Since iOS 13.0 is so low on adoption, I believe is better to keep the simpler fix, regardless of the UX issue on 13.0, avoiding possible crashes.

I will do the gutenberg side changes next, but will probably go live on WPiOS 13.6.

To test:

Update release notes:

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@etoledom etoledom added the Aztec label Oct 29, 2019
@etoledom etoledom added this to the 13.5 ❄️ milestone Oct 29, 2019
@etoledom etoledom self-assigned this Oct 29, 2019
@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Oct 29, 2019

You can trigger an installable build for these changes by visiting CircleCI here.

Copy link
Copy Markdown
Contributor

@SergioEstevao SergioEstevao left a comment

Choose a reason for hiding this comment

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

I agree, :shipit:

@etoledom etoledom force-pushed the issue/12730-interact-with-url-crash branch from 0580e21 to 888a516 Compare October 31, 2019 08:54
@etoledom etoledom changed the base branch from release/13.5 to develop October 31, 2019 08:55
@etoledom etoledom modified the milestones: 13.5 ❄️, 13.6 Oct 31, 2019
@etoledom
Copy link
Copy Markdown
Contributor Author

Thank you @SergioEstevao !
I have moved this PR to target develop. Since on 13.5 there's already a merge that will fix most of the crashes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

EXC_BREAKPOINT: textView:shouldInteractWithURL:inRange:interaction:

2 participants