Skip to content

Commit

Permalink
Merge pull request #1 from stevenpack/master
Browse files Browse the repository at this point in the history
bugfix: CreateFulfillmentOrder prefixes with DisplayableOrder properties. MWS rejects it.
  • Loading branch information
asanchezdelc committed Jan 22, 2016
2 parents c3b0b5c + dffdd3e commit fb551fe
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
node_modules
.idea
6 changes: 3 additions & 3 deletions lib/fba.js
Original file line number Diff line number Diff line change
Expand Up @@ -270,9 +270,9 @@ var calls = exports.requests = {
return new FbaOutboundRequest('CreateFulfillmentOrder', {
SellerFulfillmentOrderId: { name: 'SellerFulfillmentOrderId', required: true },
ShippingSpeedCategory: { name: 'ShippingSpeedCategory', required: true, type: 'fba.ShippingSpeedCategory' },
DisplayableOrderId: { name: 'DisplayableOrder.DisplayableOrderId', required: true },
DisplayableOrderDateTime: { name: 'DisplayableOrder.DisplayableOrderDateTime', type: 'Timestamp' },
DisplayableOrderComment: { name: 'DisplayableOrder.DisplayableOrderComment' },
DisplayableOrderId: { name: 'DisplayableOrderId', required: true },
DisplayableOrderDateTime: { name: 'DisplayableOrderDateTime', type: 'Timestamp' },
DisplayableOrderComment: { name: 'DisplayableOrderComment' },
FulfillmentPolicy: { name: 'FulfillmentPolicy', required: false, type: 'fba.FulfillmentPolicy' },
FulfillmentMethod: { name: 'FulfillmentMethod', required: false },
NotificationEmails: { name: 'NotificationEmailList.member', required: false, list: true },
Expand Down

0 comments on commit fb551fe

Please sign in to comment.