Skip to content

Conversation

@danieldudzic
Copy link
Collaborator

Description

This PR addresses the following issues:

  1. Registration was failing with VALIDATION_ERROR: Missing required fields: wooSydeCommerceId
  2. Unregistration would delete the token even when it failed, causing state inconsistency
  3. Auto-registration was creating an endless retry loop - if registration failed for any reason, the system would attempt to register on every page load, spamming PayPal's endpoint and logs with failing requests
  4. No logging made it impossible to debug failures

Future Considerations

  • If we want auto-registration, we should consider adding more robust retry logic
  • Or add a setting to disable auto-registration for debugging purposes

@github-actions
Copy link

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 v3.3.0-pr3879-19633028502-gd642e3c (built from this PR)

Login credentials:

  • Username: admin
  • Password: password

Plugin Details:

  • Version: 3.3.0-pr3879-19633028502-gd642e3c
  • Commit: d642e3c
  • Artifact: woocommerce-paypal-payments-3.3.0-pr3879-19633028502-gd642e3c

💡 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 d642e3c • Last updated: 2025-11-24T11:50:15.560Z

@stracker-phil stracker-phil changed the base branch from dev/PCP-5564-agent-inspector to PCP-4891-agentic-commerce November 25, 2025 16:44
@stracker-phil stracker-phil changed the title Agentic: Fix Registration Issues Agentic: Fix Registration Issues (5564) Nov 28, 2025
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.

3 participants