Skip to content

Conversation

@kieran-osgood-shopify
Copy link
Contributor

@kieran-osgood-shopify kieran-osgood-shopify commented Nov 27, 2025

What changes are you making?

Implements checkout.paymentMethodChangeStart event from the native libraries:
https://github.com/Shopify/checkout-sheet-kit-swift/pull/475/files
https://github.com/Shopify/checkout-sheet-kit-android/pull/425/files


PR Checklist

Important

Releasing a new version of the kit?


Tip

See the Contributing documentation for instructions on how to publish a new version of the library.

@kieran-osgood-shopify kieran-osgood-shopify requested a review from a team as a code owner November 27, 2025 19:00
@kieran-osgood-shopify kieran-osgood-shopify changed the title refactor: remove intent from event naming Feat: add checkout.paymentMethodChangeStart Nov 27, 2025
@github-actions
Copy link

github-actions bot commented Nov 27, 2025

Coverage Report

Lines Statements Branches Functions
Coverage: 82%
81.58% (226/277) 83.07% (108/130) 79.74% (63/79)

@kieran-osgood-shopify kieran-osgood-shopify force-pushed the prototype-development/issue-8681-checkout.paymentmethodchangestart2 branch from 6ee87b9 to 88d6125 Compare November 27, 2025 19:05
@objc var onLinkClick: RCTBubblingEventBlock?
@objc var onAddressChangeStart: RCTBubblingEventBlock?
@objc var onPaymentChangeIntent: RCTBubblingEventBlock?
@objc var onPaymentMethodChangeStart: RCTBubblingEventBlock?
Copy link
Contributor

Choose a reason for hiding this comment

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

do we need to add it to InlineCheckoutEventProcessor?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep! moved this back to in draft, I meant to open the android pr as non-draft

@kieran-osgood-shopify kieran-osgood-shopify force-pushed the prototype-development/issue-8681-checkout.paymentmethodchangestart2 branch from 4eb145d to f0f8bb9 Compare November 28, 2025 12:17
Base automatically changed from prototype-development/inline-android-component to prototype/development November 28, 2025 13:12
@kieran-osgood-shopify kieran-osgood-shopify marked this pull request as draft November 28, 2025 13:46
@kieran-osgood-shopify kieran-osgood-shopify changed the title Feat: add checkout.paymentMethodChangeStart #8681 add checkout.paymentMethodChangeStart Nov 28, 2025
@kieran-osgood-shopify kieran-osgood-shopify changed the title #8681 add checkout.paymentMethodChangeStart #8681 add checkout.paymentMethodChangeStart Nov 28, 2025
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the prototype-development/issue-8681-checkout.paymentmethodchangestart2 branch from c58840e to d9a3354 Compare November 28, 2025 15:44
@kieran-osgood-shopify kieran-osgood-shopify marked this pull request as ready for review November 28, 2025 18:18
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