Skip to content

Snapshot top n StakeEntries for staking rewards #598

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
Merged
tholonious merged 17 commits into from
Jul 17, 2023
Merged

Snapshot top n StakeEntries for staking rewards #598

tholonious merged 17 commits into from
Jul 17, 2023

Conversation

@tholonious
Copy link
Contributor

@tholonious tholonious commented Jul 13, 2023

No description provided.

@tholonious
Copy link
Contributor Author

tholonious commented Jul 13, 2023 •
edited
Loading

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@tholonious tholonious mentioned this pull request Jul 13, 2023
Merged
@tholonious tholonious force-pushed the Snapshot_top_n_StakeEntries_for_staking_rewards branch from 92c2d69 to 4e62dec Compare July 13, 2023 23:10
@tholonious tholonious force-pushed the Snapshot_top_n_StakeEntries_for_staking_rewards branch from 1e0b2e8 to ec89e5f Compare July 14, 2023 15:37
@tholonious tholonious force-pushed the Snapshot_top_n_StakeEntries_for_staking_rewards branch from ec89e5f to 099b5f6 Compare July 14, 2023 15:37
@tholonious tholonious requested review from AeonSw4n and lazynina July 14, 2023 18:39
@tholonious tholonious marked this pull request as ready for review July 14, 2023 18:39
@tholonious tholonious requested a review from a team as a code owner July 14, 2023 18:39
This was referenced Jul 14, 2023
Merged
Merged
@tholonious tholonious force-pushed the Snapshot_top_n_StakeEntries_for_staking_rewards branch from 171eb2f to 605d502 Compare July 14, 2023 19:19
@tholonious tholonious mentioned this pull request Jul 14, 2023
Merged
lazynina
lazynina approved these changes Jul 17, 2023
View reviewed changes
Copy link
Member

@lazynina lazynina left a comment

Choose a reason for hiding this comment

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

just one nit on the comments

@tholonious tholonious merged commit 00f9cd2 into feature/pos-staker-snapshotting-and-rewards Jul 17, 2023
@tholonious tholonious deleted the Snapshot_top_n_StakeEntries_for_staking_rewards branch July 17, 2023 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lazynina lazynina lazynina approved these changes

@AeonSw4n AeonSw4n Awaiting requested review from AeonSw4n

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tholonious @lazynina