[3.2.0] [Android] Handle geolocation requests

[markmur]

What changes are you making?

Fixes #129

sequenceDiagram
    participant Webview
    participant CheckoutSheetKit
    participant NativeModule
    participant App

    Webview->>CheckoutSheetKit: Request geolocation permissions
    CheckoutSheetKit->>NativeModule: onGeolocationShowPrompt(origin, callback)
    NativeModule->>NativeModule: Store origin and callback for later
    NativeModule->>App: Emit "geolocationRequest" event
    App->>App: Request geolocation permissions
    App->>NativeModule: Invoke native callback with result
    NativeModule->>CheckoutSheetKit: invoke the original prompt

Loading

---

PR Checklist

Important

• I've added tests to support my implementation
• I have read and agree with the Contribution Guidelines.
• I have read and agree with the Code of Conduct.
• I've updated the README.
• I have bumped the version number in the package.json file.
• I have added a Changelog entry.

[Copilot]

Copilot reviewed 6 out of 9 changed files in this pull request and generated 1 comment.

Files not reviewed (3)

• modules/@shopify/checkout-sheet-kit/android/gradle.properties: Language not supported
• sample/android/gradle.properties: Language not supported
• README.md: Evaluated as low risk

Comments suppressed due to low confidence (5)

modules/@shopify/checkout-sheet-kit/android/src/main/java/com/shopify/reactnative/checkoutsheetkit/CustomCheckoutEventProcessor.java:83

• Minor spelling mistake in the comment: 'retainGeolocationForFutureRequests' should be 'retainGeolocationForFutureRequests'.

public void onGeolocationPermissionsShowPrompt(@NonNull String origin, @NonNull GeolocationPermissions.Callback callback) {

modules/@shopify/checkout-sheet-kit/android/src/main/java/com/shopify/reactnative/checkoutsheetkit/CustomCheckoutEventProcessor.java:144

• The HashMap should specify generic types as HashMap<String, Object> for better type safety.

private Map<String, Object> populateErrorDetails(CheckoutException checkoutError) {

modules/@shopify/checkout-sheet-kit/android/src/main/java/com/shopify/reactnative/checkoutsheetkit/ShopifyCheckoutSheetKitModule.java:287

• The getConfig and setConfig methods are duplicated in the file. This seems unintentional and should be corrected.

  public void getConfig(Promise promise) {

modules/@shopify/checkout-sheet-kit/src/index.ts:78

• The type of geolocationCallback should be Maybe to match the return type of addEventListener.

private geolocationCallback: Maybe<EventSubscription>;

modules/@shopify/checkout-sheet-kit/src/index.ts:252

• The requestGeolocation method should handle cases where both permissions are denied explicitly.

const results = await PermissionsAndroid.requestMultiple([coarse, fine]);

Copilot reviewed 6 out of 10 changed files in this pull request and generated no comments.

Files not reviewed (4)

• modules/@shopify/checkout-sheet-kit/android/gradle.properties: Language not supported
• sample/android/gradle.properties: Language not supported
• modules/@shopify/checkout-sheet-kit/src/context.tsx: Evaluated as low risk
• sample/src/App.tsx: Evaluated as low risk