Skip to content

Staking: Add dest to Rewarded to aid in reward calculations#1602

Merged
rossbulat merged 6 commits into
masterfrom
rb-payee-in-rewarded
Sep 18, 2023
Merged

Staking: Add dest to Rewarded to aid in reward calculations#1602
rossbulat merged 6 commits into
masterfrom
rb-payee-in-rewarded

Conversation

@rossbulat

Copy link
Copy Markdown
Contributor

Addresses #129.

Returns Self:payee() from make_payout in a tuple alongside an imbalance & adds it to Rewarded event.

@rossbulat rossbulat requested review from a team September 17, 2023 10:27
@rossbulat rossbulat added I5-enhancement An additional feature request. T2-pallets This PR/Issue is related to a particular pallet. labels Sep 17, 2023
@rossbulat rossbulat requested a review from gpestana September 18, 2023 05:29

@kianenigma kianenigma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Would be good to add some sort of "check events" to one test that covers this and make sure it won't regress.

@rossbulat rossbulat enabled auto-merge (squash) September 18, 2023 17:17
@rossbulat rossbulat merged commit ffe5db0 into master Sep 18, 2023
@rossbulat rossbulat deleted the rb-payee-in-rewarded branch September 18, 2023 17:32
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
…tytech#1602)

Addresses paritytech#129.

Returns `Self:payee()` from `make_payout` in a tuple alongside an
imbalance & adds it to `Rewarded` event.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

I5-enhancement An additional feature request. T2-pallets This PR/Issue is related to a particular pallet.

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

4 participants