Merge pull request #331 from keep-network/update-distribution-interfaces

Updated references to keep-ecdsa contracts implementing distribution functions.

implementation/contracts/deposit/DepositRedemption.sol

@@ -6,7 +6,6 @@ import {BytesLib} from "@summa-tx/bitcoin-spv-sol/contracts/BytesLib.sol";
 import {ValidateSPV} from "@summa-tx/bitcoin-spv-sol/contracts/ValidateSPV.sol";
 import {CheckBitcoinSigs} from "@summa-tx/bitcoin-spv-sol/contracts/CheckBitcoinSigs.sol";
 import {DepositUtils} from "./DepositUtils.sol";
-import {IBondedECDSAKeep} from "../external/IBondedECDSAKeep.sol";
 import {IECDSAKeep} from "@keep-network/keep-ecdsa/contracts/api/IECDSAKeep.sol";
 import {DepositStates} from "./DepositStates.sol";
 import {OutsourceDepositLogging} from "./OutsourceDepositLogging.sol";
@@ -34,10 +33,10 @@ library DepositRedemption {
         address _tbtcTokenAddress = _d.TBTCToken;
         TBTCToken _tbtcToken = TBTCToken(_tbtcTokenAddress);
 
-        IBondedECDSAKeep _keep = IBondedECDSAKeep(_d.keepAddress);
+        IECDSAKeep _keep = IECDSAKeep(_d.keepAddress);
 
         _tbtcToken.approve(_d.keepAddress, DepositUtils.signerFee());
-        _keep.distributeERC20ToKeepGroup(_d.keepAddress, _tbtcTokenAddress, DepositUtils.signerFee());
+        _keep.distributeERC20ToMembers(_tbtcTokenAddress, DepositUtils.signerFee());
     }
 
     /// @notice Approves digest for signing by a keep

implementation/contracts/deposit/DepositUtils.sol

@@ -7,6 +7,7 @@ import {BytesLib} from "@summa-tx/bitcoin-spv-sol/contracts/BytesLib.sol";
 import {TBTCConstants} from "./TBTCConstants.sol";
 import {ITBTCSystem} from "../interfaces/ITBTCSystem.sol";
 import {IERC721} from "openzeppelin-solidity/contracts/token/ERC721/IERC721.sol";
+import {IECDSAKeep} from "@keep-network/keep-ecdsa/contracts/api/IECDSAKeep.sol";
 import {IBondedECDSAKeep} from "../external/IBondedECDSAKeep.sol";
 import {TBTCToken} from "../system/TBTCToken.sol";
 
@@ -379,7 +380,8 @@ library DepositUtils {
     /// @return             true if successful, otherwise revert
     function pushFundsToKeepGroup(Deposit storage _d, uint256 _ethValue) public returns (bool) {
         require(address(this).balance >= _ethValue, "Not enough funds to send");
-        IBondedECDSAKeep _keep = IBondedECDSAKeep(_d.keepAddress);
-        return _keep.distributeEthToKeepGroup.value(_ethValue)(_d.keepAddress);
+        IECDSAKeep _keep = IECDSAKeep(_d.keepAddress);
+        _keep.distributeETHToMembers.value(_ethValue)();
+        return true;
     }
 }

implementation/contracts/external/IBondedECDSAKeep.sol

@@ -16,15 +16,6 @@ interface IBondedECDSAKeep {
         bytes calldata _preimage
     ) external returns (bool _isFraud);
 
-    // Allow sending funds to a keep group
-    // Expected: increment their existing ETH bond
-    function distributeEthToKeepGroup(address _keepAddress) external payable returns (bool);
-
-    // Allow sending tokens to a keep group
-    // Useful for sending signers their TBTC
-    // The Keep contract should call transferFrom on the token contract
-    function distributeERC20ToKeepGroup(address _keepAddress, address _asset, uint256 _value) external returns (bool);
-
     // returns the amount of the keep's ETH bond in wei
     function checkBondAmount(address _keepAddress) external view returns (uint256);
 

implementation/package.json

@@ -21,7 +21,7 @@
   "author": "James Prestwich",
   "license": "UNLICENSED",
   "dependencies": {
-    "@keep-network/keep-ecdsa": "^0.1.1",
+    "@keep-network/keep-ecdsa": "^0.1.2",
     "bn-chai": "^1.0.1",
     "bn.js": "^4.11.8",
     "@summa-tx/bitcoin-spv-sol": "^2.1.0",

implementation/test/contracts/keep/ECDSAKeepStub.sol

@@ -46,6 +46,15 @@ contract ECDSAKeepStub is IECDSAKeep, IBondedECDSAKeep {
           emit SignatureRequested(_digest);
     }
 
+    function distributeETHToMembers() external payable {
+
+    }
+
+    function distributeERC20ToMembers(address _asset, uint256 _value) external {
+
+    }
+
+
     // Functions implemented for IBondedECDSAKeep interface.
 
     function submitSignatureFraud(
@@ -59,14 +68,6 @@ contract ECDSAKeepStub is IECDSAKeep, IBondedECDSAKeep {
        return success;
     }
 
-    function distributeEthToKeepGroup(address _keepAddress) external payable returns (bool){
-        return success;
-    }
-
-    function distributeERC20ToKeepGroup(address _keepAddress, address _asset, uint256 _value) external returns (bool){
-        return success;
-    }
-
     function checkBondAmount(address _keepAddress) external view returns (uint256){
         return bondAmount;
     }