dushkindigital / marketmap

New, clean repo for Shekhar
2 stars 0 forks source link

Unable to create eBay listing from Libereco listing #32

Open pdushkin opened 12 years ago

pdushkin commented 12 years ago

Getting server error when I try to submit a recently created libereco listing. Might be the eBay service. I tried in the AM today and again, just now.

shekhargulati commented 12 years ago

Thanks for testing Peter D. I tested with your login and got server exception but reason was not that Ebay is down. The issue was with data. You created LiberecoListing with shipping information USPS which is invalid.In my testing I use USPSMedia. After updating the shipping information when I tried to create the listing I again got server exception this time because payment information was incorrect. The error that I got was CC is no longer supported use another payment method. I used AM_EX and I was able to successfully create the listing http://cgi.sandbox.ebay.com/ws/eBayISAPI.dll?ViewItem&item=110100011297#ht_500wt_1156. I will remove the CC option and will check other values as well. Thanks for finding bug.

pdushkin commented 12 years ago

Hi Shekhar,

Great, thanks for pointing that out. I can see the submitted item now.

Moving forward, I think it makes sense to provide the full option list in the drop down. In other words, for shipping, all the shipping services in a DDL and all the shipping types in a DDL. This way we can fully test when submitting an item.

Does this make sense or do you think this is creating too much unnecessary work for you? If it seems too time-intensive for you, then I'll just use the API docs to get the various options, etc. However, it would be much easier to have it available to me when submitting the item. I'd like to try all cases for testing purposes.

What do you think?

prp commented 12 years ago

Perhaps related to this, I tried to create a new listing with a new shipping type called "FedEx". I then get the following error:

Not able to addListing Input data for tag is invalid or missing. Please check API documentation. Cause :

Is this because the shipping service names for ebay have to be from a preselected list? If so, would it be possible to present them to the user?

pdushkin commented 12 years ago

Here is useful detail on shipping:

http://developer.ebay.com/DevZone/XML/docs/WebHelp/wwhelp/wwhimpl/common/html/wwhelp.htm?context=eBay_XML_API&file=ManagingShipping-Specifying_Shipping_Costs_when_Creating_or_Revising_a_Listing.html

I think here is a bit of complexity here and it needs some thought. There seems to be a lot of options. Shekhar, is it possible for you to make those options available to us in the Web UI?

pdushkin commented 12 years ago

I have tried adding another item as "LocalPickup". I am getting this error:

Not able to addListing Input data for tag is invalid or missing. Please check API documentation. Cause :

Is this a deprecated or incorrect option?

shekhargulati commented 12 years ago

Thanks for your feedback guys. I spend time in the morning to figure out what's happening wrong. I found that there is an Ebay API called getEbayDetails which returns all the valid value of shipping service types. Below is the list of shipping service that Ebay supports. I was able to create listing for most of them except which are applicable for international shipping.

InternationalPriorityShipping ShippingMethodStandard USPSGlobalExpress ePacketChina ePacketSingapore ePacketHongKong ePacketThailand UPSGround USPSGlobalPriority USPSGround ShippingMethodExpress USPSEconomyLetter USPSPriority USPSEconomyParcel USPSAirmailLetter USPSPriorityFlatRateEnvelope USPSAirmailParcel USPSFirstClassMailInternational USPSPriorityMailSmallFlatRateBox USPSPriorityMailInternational USPSPriorityFlatRateBox USPSPriorityMailLargeFlatRateBox USPSPriorityMailInternationalFlatRateEnvelope USPSPriorityMailPaddedFlatRateEnvelope USPSPriorityMailLegalFlatRateEnvelope USPSPriorityMailInternationalSmallFlatRateBox USPSPriorityMailRegionalBoxA USPSPriorityMailInternationalFlatRateBox USPSPriorityMailRegionalBoxB USPSPriorityMailRegionalBoxC USPSPriorityMailInternationalLargeFlatRateBox USPSPriorityMailInternationalPaddedFlatRateEnvelope USPSFirstClass USPSPriorityMailInternationalLegalFlatRateEnvelope UPS3rdDay USPSExpressMailInternational UPS2ndDay USPSExpressMailInternationalFlatRateEnvelope USPSExpressMailInternationalLegalFlatRateEnvelope UPSWorldWideExpressPlus Other UPSWorldWideExpress ExpressInternationalShipping_2 ExpressInternationalShipping USPSParcel USPSMedia UPSWorldWideExpedited UPSWorldwideSaver UPSStandardToCanada LocalDelivery ShippingMethodOvernight FedExInternationalEconomy FedExInternationalPriority FedExGroundInternationalToCanada StandardInternational USPSExpressMail ExpeditedInternational USPSExpressFlatRateEnvelope USPSExpressMailLegalFlatRateEnvelope OtherInternational UPSNextDay UPSNextDayAir FedExHomeDelivery FedExExpressSaver FedEx2Day FedExPriorityOvernight FedExStandardOvernight USPSGlobalPrioritySmallEnvelope USPSGlobalPriorityLargeEnvelope UPSWorldWideExpressBox10kg UPSWorldWideExpressBox25kg UPSWorldWideExpressPlusBox10kg UPSWorldWideExpressPlusBox25kg USPSFirstClassLetter USPSFirstClassLargeEnvelop USPSFirstClassParcel EconomyShippingFromOutsideUS ExpressShippingFromOutsideUS USPSExpressMailFlatRateBox StandardShippingFromOutsideUS ExpeditedShippingFromOutsideUS UPS2DayAirAM FreightShippingInternational PromotionalShippingMethod PromotionalShippingMethod USPSExpressMailInternationalFlatRateBox USPSGlobalExpressGuaranteed FreightShipping Freight Delivery Pickup USPSFirstClassMailInternationalLetter USPSFirstClassMailInternationalLargeEnvelope USPSFirstClassMailInternationalParcel

pdushkin commented 12 years ago

Thanks Shekhar. Would it be possible to wrap this list in

shekhargulati commented 12 years ago

Sure will do it tomorrow morning..

pdushkin commented 12 years ago

OK, thanks Shekhar. I appreciate it.

On Thu, Jun 14, 2012 at 1:07 PM, Shekhar Gulati < reply@reply.github.com

wrote:

Sure will do it tomorrow morning..


Reply to this email directly or view it on GitHub: https://github.com/dushkindigital/marketmap/issues/32#issuecomment-6334601

pdushkin commented 12 years ago

I'd also like to point out that there are a number of shipping types. Currently, you only have "FLAT" available. Those shipping types must also be accounted for and tested. They are:

Calculated
CalculatedDomesticFlatInternational CustomCode Flat FlatDomesticCalculatedInternational Free Freight FreightFlat NotSpecified

Please see:

http://developer.ebay.com/devzone/xml/docs/Reference/eBay/types/ShippingTypeCodeType.html

Can you also add these options to a DDL so we can test.

Thanks Shekhar.

Regards, Peter

shekhargulati commented 12 years ago

I have added drop down for both ShippingService and ShippingType. I expect some of the options will not work as we are not sending all the required information. Please test it and see how many combinations you can use. I will spend some more time tomorrow on this.The code has been deployed on CloudFoundry.

prp commented 12 years ago

It's probably not worth spending time at this point to support all shipping services and types exhaustively. In my opinion, this can be added later on.

Cheers, Peter

On 15/06/2012 04:02, Shekhar Gulati wrote:

I have added drop down for both ShippingService and ShippingType. I expect some of the options will not work as we are not sending all the required information. Please test it and see how many combinations you can use. I will spend some more time tomorrow on this.


Reply to this email directly or view it on GitHub: https://github.com/dushkindigital/marketmap/issues/32#issuecomment-6348570

pdushkin commented 12 years ago

OK, I defer to Peter P. Let's hold off on anything more here. He is right, the priority is the MVP. But, I am glad we have a more complete sense of the shipping options. Thanks for posting them, Shekhar.

shekhargulati commented 12 years ago

I have closed this issue as we will be doing it in future milestones.

prp commented 12 years ago

I assigned this issue to a 'Phase 2' milestone, so we don't forget about this.