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

allow auction Bids to specify "exit on partial satisfaction" #7088

Closed
Chris-Hibbert opened this issue Feb 27, 2023 · 0 comments · Fixed by #7269
Closed

allow auction Bids to specify "exit on partial satisfaction" #7088

Chris-Hibbert opened this issue Feb 27, 2023 · 0 comments · Fixed by #7269
Assignees
Labels
enhancement New feature or request Inter-protocol Overarching Inter Protocol vaults_triage DO NOT USE Vaults VaultFactor (née Treasury)

Comments

@Chris-Hibbert
Copy link
Contributor

What is the Problem Being Solved?

Individual bids in the Dutch Auction (#7000) should be able to specify that their seat in the auction should be exited after successfully bidding on assets, regardless of whether they were entirely consumed. This provides support for arbitrageurs whose goal in the auction is to immediately sell the proceeds at the current market price in order to take advantage of the discount they got in the purchase.

This is in contrast with bidders whose goal is to accumulate assets at a discount, and who therefore want to let their bid ride and be applied to future rounds. (Those bidders can have an onDemand exit condition and pull assets out occasionally.)

Description of the Design

This would be another field in offerArgs when exercising the invitation.

Security Considerations

This fits within the Zoe security architecture, and is well within the ability of a contract to carry out.

Scaling Considerations

No impact.

Test Plan

Nothing outlandish.

@Chris-Hibbert Chris-Hibbert added enhancement New feature or request Inter-protocol Overarching Inter Protocol Vaults VaultFactor (née Treasury) labels Feb 27, 2023
@Chris-Hibbert Chris-Hibbert self-assigned this Feb 27, 2023
@ivanlei ivanlei added the vaults_triage DO NOT USE label Mar 2, 2023
@mergify mergify bot closed this as completed in #7269 Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Inter-protocol Overarching Inter Protocol vaults_triage DO NOT USE Vaults VaultFactor (née Treasury)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants