Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Add method updateSharedRewards for DPoS module #7715

Closed
Tracked by #7622
shuse2 opened this issue Nov 1, 2022 · 0 comments
Closed
Tracked by #7622

Add method updateSharedRewards for DPoS module #7715

shuse2 opened this issue Nov 1, 2022 · 0 comments
Assignees
Milestone

Comments

@shuse2
Copy link
Collaborator

shuse2 commented Nov 1, 2022

Description

Add updateSharedRewards according to the proposed LIP

Acceptance Criteria

  • Add unit test to cover all cases

Additional Information

  • It requires Q96 implementation
shuse2 added a commit that referenced this issue Nov 7, 2022
### What was the problem?

This PR resolves #7717 

### How was it solved?

- Add new dynamic reward module using reward module
- Update genesis block for example app
- Add small refactor to reward module to be usable from dynamic reward
module
- Add new DPoS method which is required by dynamic reward module
  - #7715 should add the implementation for `updateSharedRewards`
### How was it tested?

- Add unit tests for all the cases
@shuse2 shuse2 added this to the Sprint 82 milestone Nov 7, 2022
@Incede Incede self-assigned this Nov 13, 2022
shuse2 added a commit that referenced this issue Nov 18, 2022
### What was the problem?

This PR resolves #7715 #7788

### How was it solved?

* Fix bug in Q library
* Add method as specified

### How was it tested?

Added unit tests

Co-authored-by: shuse2 <[email protected]>
@shuse2 shuse2 closed this as completed Nov 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants