Merge remote-tracking branch 'origin/master'

Author: saleweaver

docs/config_file.rst

@@ -6,7 +6,7 @@ Config File
 An example config file is provided in this repository, it supports multiple accounts.
 The programm looks for a file called `credentials.yml`_
 
-The config is parsed by `confused`_, see their docs for more in depth information.
+The config is parsed by `confuse`_, see their docs for more in depth information.
 Search paths are:
 
 ..  code-block:: bash
@@ -72,6 +72,6 @@ References
 .. target-notes::
 
 .. _`credentials.yml`: https://github.com/saleweaver/python-amazon-sp-api/blob/master/credentials.yml
-.. _`confused`: https://confuse.readthedocs.io/en/latest/usage.html#search-paths
+.. _`confuse`: https://confuse.readthedocs.io/en/latest/usage.html#search-paths
 
 

requirements.txt

@@ -1,10 +1,10 @@
 pytest
 requests~=2.28.2
 
-boto3~=1.26.51
+boto3~=1.26.83
 
-cachetools~=5.2.1
-setuptools~=66.0.0
+cachetools~=5.3.0
+setuptools~=67.4.0
 pytz~=2022.7
 confuse~=2.0.0
 jinja2

setup.py

@@ -8,8 +8,7 @@
     install_requires=[
         "requests",
         "boto3>=1.16.39,<2",
-        "cachetools>=4.2,<5.3",
-        "pytz",
+        "cachetools>=4.2,<5.4",
         "confuse>=1.4,<2.1"
     ],
     extras_require={

sp_api/.DS_Store

@@ -156,7 +156,7 @@ def create_feed_document(self, file, content_type, **kwargs) -> ApiResponse:
         For more information, see "Usage Plans and Rate Limits" in the Selling Partner API documentation.
 
         Args:
-            file: File or File like object
+            file: File or File like object. Setting this to None will skip the upload.
             content_type: str
             body: | * REQUIRED {'description': 'Specifies the content type for the createFeedDocument operation.', 'properties': {'contentType': {'description': 'The content type of the feed.', 'type': 'string'}}, 'required': ['contentType'], 'type': 'object'}
 
@@ -168,6 +168,10 @@ def create_feed_document(self, file, content_type, **kwargs) -> ApiResponse:
             'contentType': kwargs.get('contentType', content_type)
         }
         response = self._request(kwargs.get('path'), data={**data, **kwargs})
+
+        if(file is None):
+            return response
+
         upload_data = file.read()
         try:
             upload_data = upload_data.decode('iso-8859-1')

sp_api/api/feeds/feeds.py

@@ -62,7 +62,7 @@ def get_order_metrics(self, interval: tuple, granularity: Granularity, granulari
         if granularityTimeZone:
             kwargs.update({'granularityTimeZone': granularityTimeZone})
         if 'sku' in kwargs:
-            kwargs.update({'sku': urllib.parse.quote_plus(kwargs.pop('sku'))})
+            kwargs.update({'sku': urllib.parse.quote(kwargs.pop('sku'), safe='')})
         return self._request(kwargs.pop('path'), params=kwargs)
 
     @staticmethod

sp_api/api/sales/sales.py

@@ -3,10 +3,11 @@
 import base64
 import warnings
 import functools
+from urllib import parse
 
 
 def fill_query_params(query, *args):
-    return query.format(*args)
+    return query.format(*[parse.quote(arg, safe='') for arg in args])
 
 
 def sp_endpoint(path, method='GET'):