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

fixing takeProtocolFee on honeyswap #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Luigy-Lemon
Copy link

No description provided.

@Luigy-Lemon
Copy link
Author

Luigy-Lemon commented Dec 12, 2022

Old deployment at: 0xba13a6414c677d9e7bd3c0bde5ba4055bf2e72f7

Deployed at 0x9cf1a1f584904dc4860c1b00c8fb1cb028c90c62 https://gnosisscan.io/address/0x9cf1a1f584904dc4860c1b00c8fb1cb028c90c62#code
Use 0xf26ebf172966e22e3448422969d872802326916b aragon multisig to update factory 0xA818b4F111Ccac7AA31D0BCc0806d64F2E0737D7 by calling setFeeTo() with 0x19749f4d55fbbfaf93a8c6f532ccdb20f67c9926

…ll would revert rather than return 0 - gas efficiency improvements
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.

1 participant