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

Add a configurable claim fee #67

Merged
merged 1 commit into from
Oct 31, 2024
Merged

Add a configurable claim fee #67

merged 1 commit into from
Oct 31, 2024

Conversation

apbendi
Copy link
Collaborator

@apbendi apbendi commented Oct 23, 2024

The claim fee is intentionally set as a fixed amount. The primary purpose of this
fee is to act as a deterrent to "deposit sybiling", whereby a staker might spread
funds out across many deposits in an attempt to reduce the liklihood or frequency
of being "bumped" if his earning power decreases. Allowing the DAO to set a small,
fixed fee would decrease the effective yield of small deposits without materially
impacting a normal user.

@apbendi apbendi force-pushed the claim-fee branch 2 times, most recently from efdc337 to 03e00c3 Compare October 30, 2024 15:23
@apbendi apbendi marked this pull request as ready for review October 30, 2024 15:24
Copy link
Collaborator

@alexkeating alexkeating left a comment

Choose a reason for hiding this comment

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

Looking great all small nits

test/GovernanceStaker.t.sol Outdated Show resolved Hide resolved
test/GovernanceStaker.t.sol Outdated Show resolved Hide resolved
test/harnesses/GovernanceStakerHarness.sol Show resolved Hide resolved
test/harnesses/GovernanceStakerHarness.sol Show resolved Hide resolved
src/GovernanceStaker.sol Show resolved Hide resolved
The claim fee is intentionally set as a fixed amount. The primary purpose of this
fee is to act as a deterrent to "deposit sybiling", whereby a staker might spread
funds out across many deposits in an attempt to reduce the liklihood or frequency
of being "bumped" if his earning power decreases. Allowing the DAO to set a small,
fixed fee would decrease the effective yield of small deposits without materially
impacting a normal user.
Copy link

Coverage after merging claim-fee into main will be

99.28%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   BinaryEligibilityOracleEarningPowerCalculator.sol100%100%100%100%
   DelegationSurrogate.sol100%100%100%100%
   DelegationSurrogateVotes.sol100%100%100%100%
   GovernanceStaker.sol98.97%100%97.96%99.10%295–296

@apbendi apbendi requested a review from alexkeating October 31, 2024 14:20
@apbendi apbendi merged commit 63bbf23 into main Oct 31, 2024
4 checks passed
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.

2 participants