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

[Feature Request]: Wrapper API request body formed into JSON format #6

Open
MattMoszczynski opened this issue Aug 18, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@MattMoszczynski
Copy link

MattMoszczynski commented Aug 18, 2022

Feature Description

I have notied that bodies of each request sent to UPS API are formed into XML fomat, but looking through latest documentation of UPS API it suggests to send body requests in JSON format. I think this also cause a The request is well formed but the request is not valid (10002) error getting thrown whenever I try to use the wrapper. Would it be possible in the future to add functionality for wrapper to form requests body in JSON format?

Is this feature valuable for other users as well and why?

The UPS API has changed and will be changing. XML format is also getting slowly replaced with JSON format. I believe it's important to implement JSON based requests for the wrapper.

@MattMoszczynski MattMoszczynski changed the title [Feature Request]: Wrapper API requests formed into JSON format [Feature Request]: Wrapper API request body formed into JSON format Aug 18, 2022
@rawilk rawilk added the enhancement New feature or request label Nov 3, 2022
@rawilk
Copy link
Owner

rawilk commented Nov 3, 2022

This is definitely something that should be done to this package at some point. I'm personally using EasyPost to ship packages now, so I'm not really using this package in my own projects anymore so I'm not sure when I'll have a chance to work on that.

@rawilk rawilk pinned this issue Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants