Skip to content

Omit Google Pay billingAddressParameters when billing is not required - #987

Merged
Max Harrison (maxharrison) merged 2 commits into
masterfrom
max/gpay-omit-billing-address-parameters-v2
Aug 24, 2026
Merged

Omit Google Pay billingAddressParameters when billing is not required#987
Max Harrison (maxharrison) merged 2 commits into
masterfrom
max/gpay-omit-billing-address-parameters-v2

Conversation

@maxharrison

@maxharrison Max Harrison (maxharrison) commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

buildPaymentRequest sent billingAddressParameters even when billingAddressRequired was false. Google ignores those parameters, but the request still claimed a FULL billing address format that the merchant did not request.

The request now omits billingAddressParameters unless billing collection is enabled. This does not change the Google Pay sheet, and it matches the Android request for the disabled case.

Stack: #985 adds golden request tests on top of this PR.

The web request always sent billingAddressParameters, even when
billingAddressRequired was false. Google ignores the field in that case,
so the payment sheet does not change, but the request claimed a FULL
address format that we never asked for.

Omitting the field makes the request say only what it means, and matches
the Android SDK.
@maxharrison
Max Harrison (maxharrison) requested a review from a team as a code owner August 20, 2026 16:03
@changeset-bot

changeset-bot Bot commented Aug 20, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 20ba221

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@evervault/ui-components Minor
@evervault/ui-components-e2e-tests Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@maxharrison

Copy link
Copy Markdown
Contributor Author

Should we release a changeset with this?

@ana-maksimovskikh ana-maksimovskikh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM - thank you 🙏

@maxharrison
Max Harrison (maxharrison) added this pull request to the merge queue Aug 24, 2026
Merged via the queue into master with commit 92bd94b Aug 24, 2026
24 of 25 checks passed
@maxharrison
Max Harrison (maxharrison) deleted the max/gpay-omit-billing-address-parameters-v2 branch August 24, 2026 13:28
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