-
Notifications
You must be signed in to change notification settings - Fork 104
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(node): Modify rent reward distributions #3686
Conversation
and adjustments
add test case for disbersments from rent pool
add checking to the wait test case that rent goes to the pool
Except comments above, lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work
Btw, still maybe consider at least migrations (try-runtime) check for pool account existence? |
@CodiumAI-Agent /review |
PR Analysis
PR Feedback💡 General suggestions: The PR seems to be well-structured and follows good coding practices. However, it would be beneficial to include more detailed comments explaining the logic behind the changes, especially for complex functions. This would make it easier for other developers to understand the code and contribute in the future. 🤖 Code feedback:
✨ Usage guide:Overview:
With a configuration file, use the following template:
See the review usage page for a comprehensive guide on using this tool. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As it was said above: awesome
@ekovalev still asking for review, please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good enough to me. There is some space for optimization but it can be dealt with separately if becomes to much of a nuisance.
Conflicts: pallets/gear/src/lib.rs pallets/gear/src/pallet_tests.rs pallets/staking-rewards/src/tests.rs runtime/vara/src/migrations.rs
Resolves #3560 . Implements a part of the rent proposal for wait-lists.
create rent pool
add test runtime with two block producers
rent_pool_disbursments_work
test caserents for wait-list, mailbox, reservation, delayed sending are transferred to the rent pool
create rent pool accounts on the networks before deploying this feature
@gear-tech/dev