-
Notifications
You must be signed in to change notification settings - Fork 43
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
Add sweepTokens() function #44
Comments
Issue Status: 1. Open 2. Started 3. Submitted 4. Done This issue now has a funding of 600.0 DAI (600.0 USD @ $1.0/DAI) attached to it as part of the rtoken-project fund.
|
PLEASE NOTE: the bounty amount is 300 DAI, not 600 DAI |
Not sure what you mean? |
I've implemented the sweepTokens and tokenManager functionality as described, I did not write tests though. |
Issue Status: 1. Open 2. Started 3. Submitted 4. Done Workers have applied to start work. These users each claimed they can complete the work by 265 years, 7 months from now. Learn more on the Gitcoin Issue Details page. |
⚡️ A tip worth 400.00000 DAI (400.0 USD @ $1.0/DAI) has been granted to @CrystalPony for this issue from @pi0neerpat. ⚡️ The sender had the following public comments:
Nice work @CrystalPony! Your tip has automatically been deposited in the ETH address we have on file.
|
We need a way to get COMP, and other tokens out of the rDAI contract.
One possible solution:
sweepTokens(address token)
which is open for anyone to call, and sends any ERC20, excluding DAI and cDAI, to a manager account.setTokenManager(address newManager)
which is limited to only the current manager.Requirements:
Things to keep in mind:
tokenManager
account without admin powers.Bounty amount: 300 DAI
The text was updated successfully, but these errors were encountered: