Inconsistent Behavior in CheckoutDelegateWrapper Callbacks (onComplete, onCancel)

[sumeyyekazanci]

What area is the issue related to?

Checkout Sheet Kit

What version of Checkout Sheet Kit are you using?

3.0.4-3.1.2

Reproducible sample code

No response

Steps to Reproduce

• When the app state changes to inactive and then returns to the foreground, the onCancel callback does not get triggered.

• iOS 17.0 onCancel never works

• onComplete is not stable.

Expected Behavior

The onComplete, onCancel, and other related callbacks should trigger reliably when the associated events occur.

Actual Behavior

The Shopify Sheet Kit SDK does not work stably in certain scenarios, and specifically, callback functions such as onComplete and onCancel defined within the CheckoutDelegateWrapper behave inconsistently. This issue prevents users from receiving feedback during certain operations and causes problems in the application flow.

Screenshots/Videos/Log output

No response

Storefront domain

flyingtiger-tr.myshopify.com

[markmur]

Hey @sumeyyekazanci, can you provide some sample code to demonstrate how you're using the kit?

Any videos or logs would also be helpful for us to debug.

[markmur]

Here is some sample code if using SwiftUI to integrate →

checkout-sheet-kit-swift/Samples/SwiftUIExample/SwiftUIExample/CartView.swift

Lines 63 to 100 in d319147

	.sheet(isPresented: $isShowingCheckout) {
	if let url = checkoutURL {
	CheckoutSheet(checkout: url)
	/// Configuration
	.title("SwiftUI")
	.colorScheme(.automatic)
	.tintColor(UIColor(red: 0.33, green: 0.20, blue: 0.92, alpha: 1.00))
	/// Lifecycle events
	.onCancel {
	if cartCompleted {
	cartManager.resetCart()
	cartCompleted = false
	}
	isShowingCheckout = false
	}
	.onPixelEvent { event in
	switch event {
	case .standardEvent(let event):
	print("WebPixel - (standard)", event.name!)
	case .customEvent(let event):
	print("WebPixel - (custom)", event.name!)
	}
	}
	.onLinkClick { url in
	if UIApplication.shared.canOpenURL(url) {
	UIApplication.shared.open(url)
	}
	}
	.onComplete { checkout in
	print("Checkout completed - Order id: \(String(describing: checkout.orderDetails.id))")
	cartCompleted = true
	}
	.onFail { error in
	print(error)
	}
	.edgesIgnoringSafeArea(.all)
	.accessibility(identifier: "CheckoutSheet")

You can also configure the logLevel of the kit to .all to output more information.

ShopifyCheckoutSheetKit.configuration.logLevel = .all

[plump-pinniped]

Hey I was also running into this issue. For me, the issue happened when the view hosting the CheckoutSheet updated. I think it's because the CheckoutWebViewController only has a weak ref to the delegate object. When the CheckoutSheet is updated, a new delegate is instantiated so the old one is deallocated. Setting a breakpoint here, delegate was nil. I guess you'd have to update the delegate somehow in updateUIViewController

I did a workaround by making the host view Equatable based off of url so that the CheckoutSheet doesn't update. That fixed the issue for me