Eric Blade

callEndpoint saveRaw and saveParsed options are never hit if the server errors

1

I would sort of expect that they would probably both hit, saving the error data to the file. There's some inconsistency here, particularly when dealing with parseEndpoint vs callEndpoint --...

create a mock mws server code to test mws-advanced (and mws-simple) without live data

1

We should be able to use the example outputs from the MWS documentation to put together a library of data to test many things without going live. We can also...

I am presently using GetCompetitivePricingForASIN without a wrapper :-S

I have a note in some of my code that uses it, that the library should be fixed to always return an array at the top level of it's return....

do something expected when calling init() with no keys, or when making calls with no keys available, such as skipping init()

1

Right now, it just spits errors somewhere deep in the code, where it shouldn't even get to, if you haven't supplied keys to the lib.

getMatchingProductForId returns extremely long floats that look wrong in packageDimensions and itemDimensions

10

Sometimes, getMatchingProductForId returns itemDimensions in extremely long floats-as-strings. ~~It's unclear if this is coming from Amazon directly, or if it's something that is occurring as the data gets handed between...

With Feed support now functioning at the most basic level, this leaves a lot of possibilities for dynamic feed generation. What I'm toying with right now, is creating a class...

cache: replace with a middleware-type solution

1

So, in hindsight, while I needed the functionality of the cache system that was implemented, implementing it in the way that I did was pretty dumb. So, this is a...

GetMyFeesEstimate is expecting a list of keys that would look a bit in code like: ```` // FeesEstimateRequestList.FeesEstimateRequest.1= // { // IdType: 'ASIN', // type: 'MarketplaceType', values: ['ASIN', 'SellerSKU'], //...

add ability to "get all" of every type of function that provides a *ByNextToken equivalent

3

We should add a parameter to all the base functions that also have *ByNextToken versions, that will allow the MWS lib to go and gather up *ALL* of the data...