Skip to content
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.

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

PackageResults does not work as expected #3

Closed
DemigodCode opened this issue Nov 2, 2023 · 3 comments
Closed

PackageResults does not work as expected #3

DemigodCode opened this issue Nov 2, 2023 · 3 comments

Comments

@DemigodCode
Copy link

The PackageResults Property of ShipmentResults can be an array of PackageResults or just the properties of an PackageResults-Object

Single:
"PackageResults": { "TrackingNumber": "1ZXXXXXXXXXXXXXXXX", "ServiceOptionsCharges": { "CurrencyCode": "USD", "MonetaryValue": "0.00" }, "ShippingLabel": { "ImageFormat": { "Code": "ZPL", "Description": "ZPL" }, "GraphicImage": "" } }

Multiple:

"PackageResults": [ { "TrackingNumber": "1ZXXXXXXXXXXXXXXXX", "ServiceOptionsCharges": { "CurrencyCode": "USD", "MonetaryValue": "0.00" }, "ShippingLabel": { "ImageFormat": { "Code": "ZPL", "Description": "ZPL" }, "GraphicImage": "" } }, { "TrackingNumber": "1ZXXXXXXXXXXXXXXXX", "ServiceOptionsCharges": { "CurrencyCode": "USD", "MonetaryValue": "0.00" }, "ShippingLabel": { "ImageFormat": { "Code": "ZPL", "Description": "ZPL" }, "GraphicImage": "" } } ]

The swagger generated classes do not support this and currently only array as PackageResults are supported.

@abolabo
Copy link
Collaborator

abolabo commented Nov 8, 2023

in the v1.0.2 i see arrays on containers in the response. Looks like ups dev team fixed it.
Please check and let us know.

@abolabo abolabo closed this as completed Nov 8, 2023
@DemigodCode
Copy link
Author

Yes, there are containers, but these containers are empty, cause if you create one single package in a shipment, it returns the package property instead of an array of packages. It's still not working. Same error as before.

@DemigodCode
Copy link
Author

UPS-API/api-documentation#3 (comment)

I have an ongoing discussion with UPS about that and others too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants