-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Including in the documentation how the swap fee is enforced in Uniswap V2 #5185
Conversation
Gatsby Cloud Build Reportethereum-org-website-dev 🎉 Your build was successful! See the Deploy preview here. Build Details🕐 Build time: 6m PerformanceLighthouse report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for spotting and fixing this @astutejoe ✌️.
Be sure to join the discord if you are interested in contributing further to the project or have any questions for the team. And remember to claim your POAP also 🥳! We're currently working on our 2022 POAPs and they'll be released soon.
@all-contributors please add @astutejoe for docs |
I've put up a pull request to add @astutejoe! 🎉 |
Hey, that's very nice of you @minimalsm !! I appreciate it! I didn't know about the POAPs, that's great too! I'll make sure to find more places where I can meaningfully contribute :) |
Description
Currently, the documentation only highlights how we can quote a token amount while accounting for the swap fees, we also mention how the protocol earns fees but it was never explained how the liquidity pool earns exchange fees, understandably so as the code for that is rather convoluted, which is also why it's especially important that it's mentioned in this breakdown.
English is not my native language and this is rather a complex topic/calculation so I'd be happy to include any suggested changes.
Related issue: #5186