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: Protocol Fees Tracker 2: PriceImprovement over Quotes #2287

Closed
4 of 9 tasks
sunce86 opened this issue Jan 15, 2024 · 4 comments
Closed
4 of 9 tasks

feat: Protocol Fees Tracker 2: PriceImprovement over Quotes #2287

sunce86 opened this issue Jan 15, 2024 · 4 comments
Labels
E:4.2 Protocol Fee Implementation See https://github.com/cowprotocol/pm/issues/29 for details

Comments

@sunce86
Copy link
Contributor

sunce86 commented Jan 15, 2024

Problem

In previous stage we implemented protocol fees for out-of-market limit orders: #2114

In next stage we want to charge the fee from in-market orders (both market and limit orders).

Tasks

@fleupold
Copy link
Contributor

WARNING: this potentially requires A LOT of time. It would require allowing an order to have both signed user_fee and surplus_fee (through which we model protocol fees).

Maybe we should make this dependent on having shipped fee = 0 before then to avoid this work? (cc @olgafetisova & @harisang). Right now an order can either have a fixed user signed fee or a fee that is taken from the surplus, but never both.

@sunce86
Copy link
Contributor Author

sunce86 commented Jan 16, 2024

Do we also need to make sure integrators are not sending us the orders with user signed fee != 0? Otherwise, if we do not support protocol fees for those, then those orders will not pay protocol fees?

@sunce86
Copy link
Contributor Author

sunce86 commented Feb 7, 2024

Do we also need to make sure integrators are not sending us the orders with user signed fee != 0? Otherwise, if we do not support protocol fees for those, then those orders will not pay protocol fees?

decided with @olgafetisova that it is fine to wait for fee=0 before releasing price improvements over quotes.

squadgazzz added a commit that referenced this issue Feb 12, 2024
# Description
Closes task n3 from the
#2287

## How to test
TBD

---------

Co-authored-by: Martin Beckmann <[email protected]>
squadgazzz added a commit that referenced this issue Feb 12, 2024
@squadgazzz squadgazzz added the E:4.2 Protocol Fee Implementation See https://github.com/cowprotocol/pm/issues/29 for details label Mar 4, 2024
@squadgazzz squadgazzz self-assigned this Mar 4, 2024
@squadgazzz squadgazzz removed their assignment Mar 5, 2024
@fleupold
Copy link
Contributor

Closing in favor of individual tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E:4.2 Protocol Fee Implementation See https://github.com/cowprotocol/pm/issues/29 for details
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants