Stop including optional query parameters when value is set to None

Author: dbanty

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## 0.1.1 - 2020-03-06
 - Fix mypy issue in generated models `from_dict` with datetime or reference properties
 - Generated clients now raise an `ApiResponseError` if they receive a response that was not declared
+- Stop including optional query parameters when value is set to None
 
 ## 0.1.0 - 2020-02-28
 - Initial Release

openapi_python_client/templates/endpoint_module.pyi

@@ -47,9 +47,17 @@ def {{ endpoint.name }}(
     {% if endpoint.query_parameters %}
     params = {
         {% for parameter in endpoint.query_parameters %}
-        "{{ parameter.name }}": {{ parameter.transform() }},
+        {% if parameter.required %}
+            "{{ parameter.name }}": {{ parameter.transform() }},
+        {% endif %}
         {% endfor %}
     }
+    {% for parameter in endpoint.query_parameters %}
+    {% if not parameter.required %}
+    if {{ parameter.name }} is not None:
+        params["{{ parameter.name }}"] = {{ parameter.transform() }}
+    {% endif %}
+    {% endfor %}
     {% endif %}
 
     response = requests.{{ endpoint.method }}(