easypost-csharp icon indicating copy to clipboard operation
easypost-csharp copied to clipboard

[Feat]: Create Parameters.Options.Create to be consistent with other Parameters.Shipment.Create property types

Open chadwixk opened this issue 1 year ago • 1 comments

Feature Request Is New

  • [X] I have verified that the requested feature does not already exist or has not already been requested.

Description of the feature

Parameters.Shipment.Create.cs properties are other Parameter types, except Options which use a non-parameter type and instead use the Model.API type.

chadwixk avatar Oct 12 '24 14:10 chadwixk

Hello, thanks for reaching out!

You are correct, the Options class is one of, if not the only, response-model class that does not have a corresponding parameter class. This is largely due to the fact that a) the Options class is quite large and could potentially fall out of sync trying to maintain two versions of it, and b) there is no difference between the request/parameter version and the response/model version, so we found it largely unnecessary to duplicate.

We are currently working on adding additional elements to the Options class, including documentation for every option, and will take your design suggestion under consideration. We apologize for the confusion.

nwithan8 avatar Oct 13 '24 07:10 nwithan8

As there are so many shipment options, we don't have plans to adjust this per the above. If ever there is an option missing, one could use the AddAdditionalOption to pass something that isn't explicitly present in the lib.

Justintime50 avatar Mar 18 '25 15:03 Justintime50