Skip to content

Conversation

@bc-afeng
Copy link
Contributor

@bc-afeng bc-afeng commented Nov 3, 2025

PAYMENTS-11168

What changed?

  • Updating the order_id field for the store/order/transaction/created scope webhook event to be string.

Testing

Performing a test in integration, we can see that the order_id is actually returned as a string. So updating the docs to accurately reflect this.

{
  "producer": "stores/ktvx0elrm2",
  "hash": "0c54b57ccda4fd93c28454242c862a9e8fd526f2",
  "created_at": 1762126619,
  "store_id": "16532817",
  "scope": "store/order/transaction/created",
  "data": {
    "order_id": "100",
    "transaction_id": "80047454917",
    "type": "transaction",
    "transaction_status": "complete",
    "transaction_type": "purchase",
    "result": {
      "code": "paid",
      "message": "Paid",
      "type": "success"
    },
    "platform_transaction_id": "5292709",
    "provider_transaction_id": "80047454917"
  }
}

Release notes draft

  • Fixed an error to update the order_id field for the store/order/transaction/created scope webhook event to be string. This now accurately reflects the type that is returned in the payload.

Anything else?

ping @bc-afeng

@bc-afeng bc-afeng changed the title fix(payment): PAYMENTS-11168 Update transaction created webhook orderid to be string fix(payment): PAYMENTS-11168 Update transaction created webhook order_id to be string Nov 3, 2025
@bc-afeng bc-afeng marked this pull request as ready for review November 3, 2025 22:39
Copy link

@bc-chenli bc-chenli left a comment

Choose a reason for hiding this comment

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

Thanks for the update 👍

Copy link
Contributor

@chris-nowicki chris-nowicki left a comment

Choose a reason for hiding this comment

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

@bc-afeng bc-afeng merged commit 456541f into main Nov 4, 2025
3 checks passed
@bc-afeng bc-afeng deleted the PAYMENTS-11168 branch November 4, 2025 22:55
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.

4 participants