Skip to content

Remove retired estimates endpoints#99

Merged
pcothenet merged 3 commits into
mainfrom
pc/2_3
Jan 10, 2025
Merged

Remove retired estimates endpoints#99
pcothenet merged 3 commits into
mainfrom
pc/2_3

Conversation

@pcothenet

@pcothenet pcothenet commented Jan 10, 2025

Copy link
Copy Markdown
Contributor

What

  • Remove retired estimates API endpoints:
    • CreateEthereumEstimateRequest
    • CreateShippingEstimateRequest
    • CreateVehicleEstimateRequest

Why

  • Those endpoints were unused and have been fully retired

SDK Release Checklist

  • Have you added an integration test for the changes?
  • Have you built the package locally and made queries against it successfully?
  • Did you update the changelog?
  • Did you bump the package version?
  • If endpoints were removed, did you manually remove the corresponding files? (this should be rare)
  • For breaking changes, did you plan for the release of the new SDK versions and deploy the API to production?

@pcothenet
pcothenet requested a review from holtbp January 10, 2025 23:28
@pcothenet
pcothenet merged commit 59bbaff into main Jan 10, 2025
@pcothenet
pcothenet deleted the pc/2_3 branch January 10, 2025 23:37
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