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

feat: protorev no taker fee #6861

Merged
merged 2 commits into from
Nov 13, 2023
Merged

Conversation

czarcas7ic
Copy link
Member

Closes: #XXX

What is the purpose of the change

Right now, protrorev pays a taker fee on it's backruns. There is no need for this as this profit goes back to the protocol anyway.

@czarcas7ic czarcas7ic added the V:state/breaking State machine breaking PR label Nov 10, 2023
@github-actions github-actions bot added the C:app-wiring Changes to the app folder label Nov 10, 2023
@czarcas7ic czarcas7ic marked this pull request as ready for review November 10, 2023 19:36
Copy link
Member

@p0mvn p0mvn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, can we add my address as well? 👀

Copy link
Member

@mattverse mattverse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@ValarDragon ValarDragon merged commit 3e8f73a into main Nov 13, 2023
1 check passed
@ValarDragon ValarDragon deleted the adam/add-protorev-to-whitelist branch November 13, 2023 17:22
@github-actions github-actions bot mentioned this pull request Feb 15, 2024
@github-actions github-actions bot mentioned this pull request Mar 15, 2024
@github-actions github-actions bot mentioned this pull request May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:app-wiring Changes to the app folder V:state/breaking State machine breaking PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants