8000 add all current marketplaces, alphabetize by country code by agritheory · Pull Request #154 · python-amazon-mws/python-amazon-mws · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Jan 4, 2025. It is now read-only.

Conversation

@agritheory
Copy link
Contributor

Several more marketplaces have been added since I looked at this last:
https://docs.developer.amazonservices.com/en_US/dev_guide/DG_Endpoints.html

@codecov
Copy link
codecov bot commented Feb 27, 2020

Codecov Report

Merging #154 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #154   +/-   ##
=======================================
  Coverage   46.92%   46.92%           
=======================================
  Files           4        4           
  Lines         635      635           
  Branches       64       64           
=======================================
  Hits          298      298           
  Misses        326      326           
  Partials       11       11
Impacted Files Coverage Δ
mws/mws.py 44.82% <ø> (ø) ⬆️

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 ac6aae0...6580110. Read the comment docs.

Copy link
Member
@GriceTurrble GriceTurrble left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Personally think we should leave UK as an alias to the same destination as GB, unless we have other code passing this key to MWS itself which might cause a break.

@agritheory
Copy link
Contributor Author

Personally think we should leave UK as an alias to the same destination as GB, unless we have other code passing this key to MWS itself which might cause a break.

OK, I'll make that change (both branches)

@GriceTurrble GriceTurrble merged commit 84c1cf2 into python-amazon-mws:master Feb 27, 2020
GriceTurrble added a commit that referenced this pull request Feb 27, 2020
Due to recent regions update from #154 .
@GriceTurrble GriceTurrble mentioned this pull request Feb 27, 2020
GriceTurrble added a commit that referenced this pull request Mar 1, 2020
Due to recent regions update from #154 .
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0