You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
if the field is missing from the object in the line above instead of throwing exceptions, we call check_null_field which in turn just creates an {} for the field instead of throwing an error.
I could not figure out how optional fields are meant to work here so did not PR but to me it sounds like if it anything but an optional we should throw an exception.
Will PR a fix that i think will work but not sure.
The text was updated successfully, but these errors were encountered:
Hey,
https://github.com/palantir/conjure-python-client/blob/develop/conjure_python_client/_serde/decoder.py#L50
if the field is missing from the object in the line above instead of throwing exceptions, we call check_null_field which in turn just creates an
{}
for the field instead of throwing an error.I could not figure out how optional fields are meant to work here so did not PR but to me it sounds like if it anything but an optional we should throw an exception.
Will PR a fix that i think will work but not sure.
The text was updated successfully, but these errors were encountered: