Closed bgarret closed 3 months ago
Thanks for the offer to contribute! I didn't have good instructions until just now – they're in CONTRIBUTING.md. That said, these issues are fixed and will be released in v7 soon :)
Thanks for your work, it's very very cool that you opened the development process. I skimmed the contribution docs and made a few changes in #730 , to hopefully make it clearer.
I saw you bumped the minimum version to PHP 8.2, is this a hard requirement? PHP 8.1 is still on security support until 31 Dec 2025, we haven't migrated to a newer version yet, this would prevent us from upgrading :disappointed:
Released in v7.0.0
.
Re: the PHP 8.2 situation – it would take a lot of dependency juggling (including on highsidelabs/saloon-sdk-generator
) to get this library fully PHP 8.1 compatible. If you want to submit PRs to that repo and to this one that gets things working with PHP 8.1 I'm happy to review them, but I think I'm unlikely to do that myself anytime soon.
Problem description:
Hello,
I'm implementing the Shipping V2 API and it looks like there are a few model fields missing while using the
getTracking
method :errorList
field, it should beerrors
.SellingPartnerApi\Seller\ShippingV2\Dto\Event
has an optional shipmentType field.SellingPartnerApi\Seller\ShippingV2\Dto\TrackingSummary
has an optional trackingDetailCodes field.Is there documentation on how to regenerate DTO classes? I'm open to fixing this, but I understood that the DTO classes are auto-generated and I couldn't find the process on how to do so.
Thanks again for all your work!
Error:
ErrorList
Event and TrackingSummary
Code