saloonphp / saloon

🤠 Build beautiful API integrations and SDKs with Saloon
https://docs.saloon.dev
MIT License
2.09k stars 107 forks source link

Fix | V3 - Remove array filter on multipart builder #332

Closed Sammyjo20 closed 1 year ago

Sammyjo20 commented 1 year ago

Fixes #331

Sometimes APIs will accept an empty string or null for the contents of Multipart Values. Previously if you had tried to provide an empty string, you would get an error from Guzzle. This was because of an array_filter() that I had added previously.

I have now also written a real-world test that makes an API call to our test service to ensure real uploads work.