Skip to content

Adding and updating DHL eCommerce carrier curls for Wallet and BYOCA - #228

Merged
skappa (skappaep) merged 7 commits into
EasyPost:masterfrom
hollyabrams:dhl-ecommerce-carrier-curl
Aug 5, 2026
Merged

Adding and updating DHL eCommerce carrier curls for Wallet and BYOCA#228
skappa (skappaep) merged 7 commits into
EasyPost:masterfrom
hollyabrams:dhl-ecommerce-carrier-curl

Conversation

@hollyabrams

@hollyabramsHolly Abrams (hollyabrams) commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Justintime50
Justin Hammond (Justintime50) reviewed Aug 4, 2026
View reviewed changes

@Justintime50Justin Hammond (Justintime50) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What makes DHLECS a custom workflow carrier now? Usually custom workflow carriers do not use the traditional credentials structure but this appears to still use it.

Comment threadofficial/guides/create-carrier-curls/dhlecs.sh Outdated
@hollyabrams

Holly Abrams (hollyabrams) commented Aug 4, 2026

Copy link
Copy Markdown
ContributorAuthor

Good point, it's not actually a custom workflow in the usual sense. I added carrier-specific handling because we now need it to generate two files: the BYOCA cURL and a separate Wallet cURL.

Justintime50
Justin Hammond (Justintime50) approved these changes Aug 5, 2026
View reviewed changes

@Justintime50Justin Hammond (Justintime50) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice work! 2 small formatting things

Comment threadtools/build_create_carrier_curl_requests/build_curls.py Outdated
Comment threadtools/build_create_carrier_curl_requests/build_curls.py Outdated
@skappaep
skappa (skappaep) merged commit 352e78f into EasyPost:masterAug 5, 2026
9 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raneboRane Gridley (ranebo)ranebo approved these changes
@Justintime50Justin Hammond (Justintime50)Justintime50 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@hollyabrams@ranebo@Justintime50@skappaep