Skip to content

Conversation

@mbafford
Copy link
Contributor

Carrier account type FedexMailviewAccount has been deprecated for over a year. Cleaning up example create script.

Summary: Decommission `FedexMailviewAccount`. Cleaning up trace elements.

Release Test Plan: TBD

Reviewers: #team-fedex, #oss-group!

Tags: #no_risk_assessed

Differential Revision: https://phab.easypo.net/D65687
@mbafford mbafford requested a review from a team as a code owner July 14, 2025 13:50
Copy link
Member

@Justintime50 Justintime50 left a comment

Choose a reason for hiding this comment

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

We autogenerate these curl commands based on whatever is available via the /carrier_types endpoint from the API (I believe this was done so in the last ~3 months). Its existence here means it's probably still returning from that endpoint. We'd want it remove there before we remove it here, otherwise it'll just get re-introduced at some point when we regenerate these files. If it's been removed from that endpoint, I'm all for removing it here.

@mbafford
Copy link
Contributor Author

mbafford commented Jul 21, 2025

$ ep_curl easypost-prod /v2/carrier_types | jq '.[].type' -r | grep Fed
FedexAccount
FedexCrossBorderAccount
FedexSmartpostAccount
$

It was changed to deprecated! / unavailable_for_presentation! in the monolith in https://phab.easypo.net/D65526 on 2-July-2025.

Copy link
Member

@Justintime50 Justintime50 left a comment

Choose a reason for hiding this comment

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

Perfect!

@mbafford mbafford merged commit 2812c4f into master Jul 21, 2025
17 checks passed
@mbafford mbafford deleted the FDX-83-decommission-fedexmailview branch July 21, 2025 19:35
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.

3 participants