-
Notifications
You must be signed in to change notification settings - Fork 17
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
House number is not required #23
Comments
Ooh that's new information to me. Thanks for sharing @lukas-jansen! Right now in my use case I'm splitting the address before sending it off to Sendcloud, so requiring just an address would be easier too. I'll play around with this soon™ to see if I can make this happen without causing BC breaks. Otherwise bumping a major version for this shouldn't be too much of an issue either. |
I'm using this library in the context of this plugin https://github.com/white-nl/commerce-sendcloud and I believe house_number is required for some destinations, for example Switzerland. |
Did a bit more testing, looks like sending It will also work when you don't sent the @villermen so no actual real change must be done, except maybe defaulting it to an empty string so you could skip it if you use named arguments?. |
https://api.sendcloud.dev/docs/sendcloud-public-api/parcels/operations/create-a-parcel#request-body
https://github.com/JouwWeb/sendcloud/blob/master/src/Model/Address.php#L28
As sendcloud has an automatic extraction system in place to get the house number from the address you provide, is not required to sent the house number separately and you can combine it inside the address field.
The text was updated successfully, but these errors were encountered: