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

"member" replaced by "PrepDetails" #123

Merged

Conversation

yubrew
Copy link

@yubrew yubrew commented Dec 5, 2018

In Amazon's MWS docs, it shows PrepDetailsList => PrepDetails => etc.

http://docs.developer.amazonservices.com/en_US/fba_inbound/FBAInbound_PreparationExamples.html

&InboundShipmentPlanRequestItems.member.1.SellerSKU=SKU00001
&InboundShipmentPlanRequestItems.member.1.Quantity=1
&InboundShipmentPlanRequestItems.member.1.PrepDetailsList.PrepDetails.1.PrepInstruction=Taping          
&InboundShipmentPlanRequestItems.member.1.PrepDetailsList.PrepDetails.1.PrepOwner=AMAZON
&InboundShipmentPlanRequestItems.member.2.SellerSKU=SKU00002
&InboundShipmentPlanRequestItems.member.2.Quantity=1
&InboundShipmentPlanRequestItems.member.2.PrepDetailsList.PrepDetails.1.PrepInstruction=Labeling         
&InboundShipmentPlanRequestItems.member.2.PrepDetailsList.PrepDetails.1.PrepOwner=AMAZON
&InboundShipmentPlanRequestItems.member.2.PrepDetailsList.PrepDetails.2.PrepInstruction=Taping         
&InboundShipmentPlanRequestItems.member.2.PrepDetailsList.PrepDetails.2.PrepOwner=SELLER

@hakanensari
Copy link
Member

@yubrew thanks 👍

@hakanensari hakanensari merged commit 444cb6b into lineofflight:master Dec 5, 2018
@hakanensari
Copy link
Member

@yubrew before I release this, could you confirm this works as intended? I now see "member" used in the example code section on this page. One of the two sample codes must be incorrect.

Also, if it's indeed PrepDetails, then we should update the other two actions that use the same attribute.

@yubrew
Copy link
Author

yubrew commented Dec 5, 2018

Both member and PrepDetails work. I didn't see the other examples using member before submitting this git pull. Let's stick with member.

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