Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix error when parsing response for AssignedFulfillmentOrder #235

Merged
merged 2 commits into from
Jan 4, 2023

Conversation

paulomarg
Copy link
Contributor

@paulomarg paulomarg commented Dec 20, 2022

WHY are these changes introduced?

There was an error in the AssignedFulfillmentOrder REST resource, because it was expecting the JSON response to contain a assigned_fulfillment_order field, but in reality we get back fulfillment_order.

WHAT is this pull request doing?

Enabling resources to override the expected field, and updating the auto-generated resources to fix the issue with AssignedFulfillmentOrder.

Non-automatic changes: 4042032

Type of change

  • Patch: Bug (non-breaking change which fixes an issue)

Checklist

  • I have added a changelog entry, prefixed by the type of change noted above
  • I have added/updated tests for this change

@paulomarg paulomarg requested a review from a team December 20, 2022 16:49
@paulomarg paulomarg force-pushed the fix_assigned_fulfillment_order_parse_error branch from 5bdce1f to 312b780 Compare December 20, 2022 16:52
@paulomarg paulomarg merged commit d909abf into main Jan 4, 2023
@paulomarg paulomarg deleted the fix_assigned_fulfillment_order_parse_error branch January 4, 2023 16:27
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.

2 participants