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.

Sign up for GitHub

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

Jump to bottom

consensus, mrc, sidestake: add mrc fees to staker to rewards to be allocated via sidestaking #2753

Merged
merged 2 commits into from
Mar 31, 2024
Merged

consensus, mrc, sidestake: add mrc fees to staker to rewards to be allocated via sidestaking #2753

merged 2 commits into from
Mar 31, 2024

Commits on Mar 31, 2024

  1. Add listmandatorysidestakes rpc

    @jamescowens
    jamescowens committed Mar 31, 2024
    Configuration menu
    Copy the full SHA
    b2af56a View commit details
    Browse the repository at this point in the history

  2. Add missing mrc fees to reward used for SplitCoinStakeOutput 

    In V13 testing with simultaneous mrcs and mandatory sidestakes,
stakes were being rejected because the calculated total reward
to the staker did not include mrc fees. This apparently has been
an existing omission, but since sidestakes were not validated it
went undetected. Now that mandatory sidestakes are validated in
terms of both destination and amount, they were being rejected
by the validator as being too low.

Adding the mrc fees due to the staker to the staker's reward
fixes the problem.
    @jamescowens
    jamescowens committed Mar 31, 2024
    Configuration menu
    Copy the full SHA
    a8899f5 View commit details
    Browse the repository at this point in the history