Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Add missing MarketplaceId in ListInventorySupplyResponse #305

Merged

Conversation

tusbar
Copy link
Member

@tusbar tusbar commented Feb 18, 2021

Issues in the MWS doc:

  • Specifying a MarketplaceId when querying ListInventorySupply is not US only anymore since Brexit.
  • MarketplaceId is also exposed in ListInventorySupplyByNextToken.

@tusbar tusbar added the bug Something isn't working label Feb 18, 2021
@tusbar tusbar self-assigned this Feb 18, 2021
@shortcut-integration
Copy link

This pull request has been linked to Clubhouse Story #453: Request stocks levels for each marketplace.

@codecov
Copy link

codecov bot commented Feb 18, 2021

Codecov Report

Merging #305 (0328434) into master (173a0c1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #305   +/-   ##
=======================================
  Coverage   93.66%   93.66%           
=======================================
  Files         195      195           
  Lines        1325     1325           
=======================================
  Hits         1241     1241           
  Misses         84       84           
Impacted Files Coverage Δ
...fillment-inventory/list-inventory-supply-result.js 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3325ca7...0328434. Read the comment docs.

@tusbar tusbar merged commit 6fda4e7 into master Feb 18, 2021
@tusbar tusbar deleted the tusbar/ch453/request-stocks-levels-for-each-marketplace branch February 18, 2021 12:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Development

Successfully merging this pull request may close these issues.

None yet

1 participant