fix: check for nullable or not required

Author: dtkav

openapi_python_client/parser/properties.py

@@ -63,7 +63,7 @@ def get_imports(self, *, prefix: str) -> Set[str]:
         Args:
             prefix: A prefix to put before any relative (local) module names.
         """
-        if not self.required:
+        if self.nullable or not self.required:
             return {"from typing import Optional"}
         return set()
 

openapi_python_client/templates/model.pyi

@@ -34,10 +34,10 @@ class {{ model.reference.class_name }}:
     @staticmethod
     def from_dict(d: Dict[str, Any]) -> {{ model.reference.class_name }}:
 {% for property in model.required_properties + model.optional_properties %}
-    {% if property.required %}
-        {% set property_source = 'd["' + property.name + '"]' %}
-    {% else %}
+    {% if property.nullable or not property.required %}
         {% set property_source = 'd.get("' + property.name + '")' %}
+    {% else %}
+        {% set property_source = 'd["' + property.name + '"]' %}
     {% endif %}
     {% if property.template %}
         {% from "property_templates/" + property.template import construct %}