Closed AlexisPPLIN closed 8 months ago
@AlexisPPLIN I've split the services as per your suggestion and updated the documentation to match. Can you check if I've missed anything? I've already verified this to work on production to be safe.
@villermen I've not tested this in production but LGTM :+1:
Obviously this is a breaking change for thoses using ShippingPoint
methods (even if it was broken before).
Happy new year !
I recently started using my service point contribution from
v5.1.0
, and I realized that everything was broken :face_with_head_bandage: So here is my patch:Solves
servicepoints.sendcloud.sc
URL instead of the standardparcel.sendcloud.sc
. So I added a new constantSERVICE_POINTS_BASE_URL
.service-point
->service-points
.distance
optional (null) because only returned by the api iflatitude
andlongitude
are provided.Additions
Have a nice day :wave: