New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

all fields on response message should be required (though detail_code… #9

Closed

silesky wants to merge 1 commit into ShipEngine:master from silesky:make-all-fields-on-response-message-required

silesky commented Dec 16, 2020 •

edited

Loading

if a ResponseMessage exists, all its fields should be on the object (even if one or more is nullable).


all fields on response message should be required (though detail_code…

4fef9be

… is nullable)

silesky requested review from rkrauskopf and JamesMessinger

December 16, 2020 18:17

Contributor

rkrauskopf commented Dec 17, 2020

Added in the api-definition

rkrauskopf closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment