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

Enable slippage calculations on Arbitrum and Gnosis Chain by default #447

Closed

Conversation

harisang
Copy link
Contributor

@harisang harisang commented Dec 6, 2024

This PR enables slippage calculations by default for Gnosis Chain and Arbitrum. Some preliminary testing suggests things work fine and since we want to do slippage accounting anyways, this PR addresses that

@harisang harisang requested a review from fhenneke December 6, 2024 12:12
@fhenneke
Copy link
Collaborator

fhenneke commented Dec 6, 2024

If I remember correctly, the slippage query does not handle protocol fees (and network fees) correctly. Or rather, it handles those fees as if they were slippage. Which does not work correctly with the rest of the code which treats network and partner fees separately.

Did you see that in testing?

@fhenneke
Copy link
Collaborator

fhenneke commented Dec 9, 2024

The required change to dune queries is cowprotocol/dune-queries#87.

@harisang
Copy link
Contributor Author

Closing as the change is implemented in PR #454

@harisang harisang closed this Dec 13, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Dec 13, 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.

2 participants