Skip to content

feat: Add ComponentsBillingAddressBridge for RN interop (ACC-7169)#1727

Merged
OnurVar merged 1 commit intobn/feature/checkout-componentsfrom
ov/feat/ACC-7169-billing-address-bridge
May 4, 2026
Merged

feat: Add ComponentsBillingAddressBridge for RN interop (ACC-7169)#1727
OnurVar merged 1 commit intobn/feature/checkout-componentsfrom
ov/feat/ACC-7169-billing-address-bridge

Conversation

@OnurVar
Copy link
Copy Markdown
Contributor

@OnurVar OnurVar commented Apr 30, 2026

Description

ACC-7169

Adds ComponentsBillingAddressBridge under @_spi(PrimerInternal) so the React Native SDK can dispatch a billing-address client-session action without a public master API. Wraps the existing internal ClientSessionActionsModule.updateBillingAddressViaClientSessionActionWithAddressIfNeeded and validates that the address has at least one non-empty field with a valid CountryCode.

Mirrors the precedent set by ComponentsAnalyticsLoggingBridge (#1668).

This supersedes the public RawDataManager.setBillingAddress API merged in #1707, which will be reverted in a follow-up PR after M2 and M3 of the bridge migration land.

Manual Testing

End-to-end verified via the RN example app (primer-io/primer-sdk-react-native#351) on iOS simulator: dispatched a hardcoded PrimerAddress through the bridge; onClientSessionUpdate confirmed the server echoed back the new address.

7/7 unit tests pass: Tests/Primer/CheckoutComponents/Bridge/ComponentsBillingAddressBridgeTests.

Contributor Checklist

  • All status checks have passed prior to code review
  • I have added unit tests to a reasonable level of coverage where suitable
  • I have added UI tests to new user flows, if applicable
  • I have manually tested newly added UX
  • I have open a documentation PR, if applicable

Reviewer Checklist

  • I have verified that a suitable set of automated tests has been added
  • I have verified that the title prefix aligns to the code changes + whether a release is expected after merging the PR
  • I have verified the documentation PR aligns with this PR, if applicable

Before Merging

  • If introducing a breaking change, I have communicated it internally
  • Any related documentation PRs are ready to merge

@OnurVar OnurVar requested a review from a team as a code owner April 30, 2026 21:52
@github-actions
Copy link
Copy Markdown
Contributor

Warnings
⚠️ Please assign someone aside from CODEOWNERS (@checkout-pci-reviewers) to review this PR.

Generated by 🚫 Danger Swift against 9a87e5e

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

@OnurVar OnurVar merged commit ea67c1b into bn/feature/checkout-components May 4, 2026
11 of 13 checks passed
@OnurVar OnurVar deleted the ov/feat/ACC-7169-billing-address-bridge branch May 4, 2026 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants