Skip to content

Fix the API response keys of the "totals" property (6272)#4310

Merged
stracker-phil merged 7 commits into
dev/PCP-4891-agentic-commercefrom
dev/PCP-6272-response-keys-for-totals
Apr 30, 2026
Merged

Fix the API response keys of the "totals" property (6272)#4310
stracker-phil merged 7 commits into
dev/PCP-4891-agentic-commercefrom
dev/PCP-6272-response-keys-for-totals

Conversation

@stracker-phil

Copy link
Copy Markdown
Collaborator

Description

The CartResponse.totals specs changed since we implemented the first version of the API, we need to rename a few keys:

Old key Correct new key
.tax_total .tax
.item_total .subtotal
.amount .total

@stracker-phil
stracker-phil changed the base branch from dev/develop to dev/PCP-4891-agentic-commerce April 28, 2026 14:00
@github-actions

github-actions Bot commented Apr 28, 2026

Copy link
Copy Markdown

Test using WordPress Playground

The changes in this pull request can be previewed and tested using a WordPress Playground instance.
WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser.

🔗 Test this pull request with WordPress Playground

What's included:

  • ✅ WordPress (latest)
  • ✅ WooCommerce (latest)
  • ✅ PayPal Payments plugin v4.0.2-pr4310-25109675313-g1f94ce3 (built from this PR)

Login credentials:

  • Username: admin
  • Password: password

Plugin Details:

  • Version: 4.0.2-pr4310-25109675313-g1f94ce3
  • Commit: b575b70
  • Artifact: woocommerce-paypal-payments-4.0.2-pr4310-25109675313-g1f94ce3

💡 The demo environment resets each time you refresh. Perfect for testing!

🔄 This link updates automatically with each new commit to the PR.

⚠️ This URL is valid for 30 days from when this comment was last updated.


🤖 Auto-generated for commit b575b70 • Last updated: 2026-04-29T12:56:41.759Z

@danieldudzic danieldudzic left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@stracker-phil
stracker-phil merged commit 3124a13 into dev/PCP-4891-agentic-commerce Apr 30, 2026
10 of 11 checks passed
@stracker-phil
stracker-phil deleted the dev/PCP-6272-response-keys-for-totals branch April 30, 2026 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants