Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: bump cow-sdk and api errors #149

Merged
merged 1 commit into from
Apr 3, 2024
Merged

chore: bump cow-sdk and api errors #149

merged 1 commit into from
Apr 3, 2024

Conversation

mfw78
Copy link
Contributor

@mfw78 mfw78 commented Apr 2, 2024

Description

Updates cow-sdk which contains changes related to the API errors that the watch-tower may encounter.

Changes

  • Fix errors so that those that weren't generated by the open api generator previously are handled idiomatically.
  • Adds error handling for new type TooMuchGas - likely to be encountered when there is a bad order type that is far too inefficient for gas settlement.

How to test

  1. Verify that API errors are used from generated code.

@mfw78 mfw78 requested review from anxolin and a team April 2, 2024 11:13
@mfw78 mfw78 self-assigned this Apr 2, 2024
Copy link

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: npm/@cowprotocol/[email protected]

View full report↗︎

@mfw78 mfw78 merged commit 774ed3e into main Apr 3, 2024
4 checks passed
@mfw78 mfw78 deleted the cow-sdk-bump branch April 3, 2024 11:40
@github-actions github-actions bot locked and limited conversation to collaborators Apr 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants