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: dynamic default split #174

Merged
merged 4 commits into from
Jan 8, 2025
Merged

Conversation

0xrajath
Copy link
Contributor

@0xrajath 0xrajath commented Jan 7, 2025

Motivation

We need to support the default split dynamically in the Rewards calculation since it can be set by governance. Currently it is hardcoded to 10%.

Tests

Screenshot 2025-01-08 at 4 03 28 PM

@0xrajath 0xrajath self-assigned this Jan 7, 2025
@0xrajath 0xrajath force-pushed the rajath/dynamic-default-split branch from 9472987 to affc5fb Compare January 7, 2025 21:08
@0xrajath 0xrajath marked this pull request as ready for review January 8, 2025 17:13
@0xrajath 0xrajath requested a review from a team as a code owner January 8, 2025 17:13
@0xrajath 0xrajath requested a review from seanmcgary January 8, 2025 17:13
internal/config/config.go Outdated Show resolved Hide resolved
pkg/rewards/5_goldRfaeStakers.go Outdated Show resolved Hide resolved
pkg/rewards/8_goldOperatorODRewardAmounts.go Outdated Show resolved Hide resolved
pkg/rewards/8_goldOperatorODRewardAmounts.go Outdated Show resolved Hide resolved
pkg/rewards/9_goldStakerODRewardAmounts.go Outdated Show resolved Hide resolved
pkg/rewards/9_goldStakerODRewardAmounts.go Outdated Show resolved Hide resolved
@0xrajath 0xrajath force-pushed the rajath/dynamic-default-split branch from 62254f3 to 0300a41 Compare January 8, 2025 20:38
@seanmcgary seanmcgary requested a review from ypatil12 January 8, 2025 21:36
@0xrajath 0xrajath merged commit 319d9b7 into testnet Jan 8, 2025
7 checks passed
@0xrajath 0xrajath deleted the rajath/dynamic-default-split branch January 8, 2025 22:07
seanmcgary pushed a commit that referenced this pull request Jan 8, 2025
# Motivation
We need to support the default split dynamically in the Rewards
calculation since it can be set by governance. Currently it is hardcoded
to 10%.
seanmcgary pushed a commit that referenced this pull request Jan 14, 2025
# Motivation
We need to support the default split dynamically in the Rewards
calculation since it can be set by governance. Currently it is hardcoded
to 10%.
seanmcgary pushed a commit that referenced this pull request Jan 14, 2025
# Motivation
We need to support the default split dynamically in the Rewards
calculation since it can be set by governance. Currently it is hardcoded
to 10%.
seanmcgary pushed a commit that referenced this pull request Jan 16, 2025
# Motivation
We need to support the default split dynamically in the Rewards
calculation since it can be set by governance. Currently it is hardcoded
to 10%.
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.

3 participants