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.

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: add BatchBadDebtRepay steward contract #4

Open
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

TepNik
Copy link

@TepNik TepNik commented Jan 14, 2025

Added a new contract BatchRepayBadDebtSteward that can be used to repay the bad debts of users. It processes users in batches.

It has two functions:

  1. batchRepayBadDebt - Used to repay users' bad debts
  2. getBadDebtAmount - Used to fetch the amount of bad debts of users

@TepNik TepNik requested a review from sakulstra January 14, 2025 11:07
.gitmodules Outdated Show resolved Hide resolved
.prettierrc Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
src/maintenance/BatchRepayBadDebtSteward.sol Outdated Show resolved Hide resolved
src/maintenance/BatchRepayBadDebtSteward.sol Outdated Show resolved Hide resolved
src/maintenance/BatchRepayBadDebtSteward.sol Outdated Show resolved Hide resolved
src/maintenance/BatchRepayBadDebtSteward.sol Show resolved Hide resolved

/* INTERNAL FUNCTIONS */

function _msgSender() internal view override(OzCommonContext, OzContext) returns (address) {
Copy link
Contributor

Choose a reason for hiding this comment

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

this is a bit ugly, let me see if we can get rid of the two contexts

Copy link
Author

Choose a reason for hiding this comment

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

The problem here is that the OwnableWithGuardian and the AccessControl contracts have a different Context contract.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants