Skip to content

feat: add transactionId support to GooglePayRequest - #1731

Merged
saralvasquez merged 1 commit into
mainfrom
transaction-id-google
Oct 21, 2026
Merged

saralvasquez merged 1 commit into
mainfrom
transaction-id-google

Conversation

@saralvasquez

Copy link
Copy Markdown
Contributor

Summary of changes

  • Google requires merchants to pass a transactionId in TransactionInfo for reconciliation during promotional campaigns, but the SDK had no way to set it. Adds an optional transactionId property that is serialized into the transactionInfo JSON sent to the Google Pay API.

AI Usage

Which AI Agent Was Used?

  • Copilot
  • Claude
  • Other (Type Name Here)

How was AI used?
Used claude to update tests according to the addition of this field and had it write the changelog entry and kdoc comment

Estimated AI Code Contribution

  • less than 30%
  • 30 - 60%
  • 60 - 100%

Checklist

  • Added a changelog entry
  • Tested and confirmed payment flows affected by this change are functioning as expected

Authors

Google requires merchants to pass a transactionId in TransactionInfo
for reconciliation during promotional campaigns, but the SDK had no
way to set it. Adds an optional transactionId property that is
serialized into the transactionInfo JSON sent to the Google Pay API.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@saralvasquez
saralvasquez requested a review from a team October 20, 2026 21:49
@saralvasquez
saralvasquez requested a review from a team as a code owner October 20, 2026 21:49
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities found.

Scanned Files

None

@agedd agedd 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.

🚀

@saralvasquez
saralvasquez merged commit 0519db9 into main Oct 21, 2026
19 of 20 checks passed
@saralvasquez
saralvasquez deleted the transaction-id-google branch October 21, 2026 18:07
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.

3 participants