Skip to content

Various fixes required to pass certification#8

Merged
colinmollenhour merged 13 commits intomainfrom
certification-fixes
Feb 7, 2026
Merged

Various fixes required to pass certification#8
colinmollenhour merged 13 commits intomainfrom
certification-fixes

Conversation

@bob2021
Copy link
Copy Markdown
Collaborator

@bob2021 bob2021 commented Feb 5, 2026

This MR addresses issues discovered during FedEx certification testing, including schema updates, generator improvements, and fixes for trade document upload endpoints.

Upstream Schema Updates

  • Updated all OpenAPI schemas from FedEx's latest specifications
  • Changes to account registration, consolidation, rates/transit times, and ship APIs

Ship V1

  • Remove totalWeight from RequestedShipment.required
  • Remove accountNumber from ResponsiblePartyParty.required
  • Remove commercialInvoice from CustomsClearanceDetail.required
  • Remove tinType from TaxpayerIdentification.required
  • Fix TransactionCreateShipmentOutputVO.alerts anyOf definition that caused fallback to associative array

Trade Documents Upload V1

  • File attachments now include proper filenames in Content-Disposition headers
  • Multi-file uploads handled as separate multipart entries instead of JSON arrays
  • Add test covering uploadEtdFiles, imageUploadServiceInfo, uploadMultiEtDfiles

Generator Improvements

  • Add delete-array-values action type for schema modifications to avoid full replace
  • Fix inline object schema collision: inline objects with property names matching existing components no longer silently overwrite the actual component schema. These are now extracted as Contact_2, etc and de-duplicated when possible.

Copy link
Copy Markdown
Contributor

@colinmollenhour colinmollenhour left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@colinmollenhour colinmollenhour merged commit 6a71eee into main Feb 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants