Skip to content

Conversation

clindsay3
Copy link
Collaborator

@clindsay3 clindsay3 commented Jul 15, 2025

Motivation

The existing mechanism for presenting an alert will not work on macOS.

Description

Switch from using UIKit to present an alert, to using SwiftUI.

Follow-up on #5238

I haven't actually been able to test this, because I cannot figure out how to get this deep link testing code to activate. I tried creating a deep link button in the Paywalls web interface, and it didn't seem to like that. @joshdholtz any advice on how to test this?

Here's the error I got:

Failed to open URL myscheme://deeplinktest: Error Domain=NSOSStatusErrorDomain Code=-10814 "(null)" UserInfo={_LSLine=279, _LSFunction=-[_LSDOpenClient openURL:fileHandle:options:completionHandler:]}
ERROR: Failed to open URL as deep link: myscheme://deeplinktest

@clindsay3 clindsay3 requested review from joshdholtz, fire-at-will and a team July 15, 2025 05:53
Copy link
Member

@ajpallares ajpallares left a comment

Choose a reason for hiding this comment

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

I think it makes sense!

As it only affects the Paywall tester app, could you update the PR's title to make it clear? (e.g. "Use SwiftUI to present an alert in Paywall Tester")

@clindsay3 clindsay3 changed the title Use SwiftUI to present an alert instead of UIKit. Use SwiftUI instead of UIKit to present an alert in Paywall Tester app Jul 16, 2025
@clindsay3
Copy link
Collaborator Author

I tested triggering this alert manually just to sanity test it's displaying properly, and it is. Since the logic around triggering it is unchanged, even though I couldn't actually trigger it I'm going to go ahead and merge.

@clindsay3 clindsay3 merged commit 6b3c8d6 into main Jul 16, 2025
12 checks passed
@clindsay3 clindsay3 deleted the cross-platform-alert branch July 16, 2025 03:57
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.

2 participants