Small Fixes

Changed: 1. OpenZeppelin from 1.9.0 to 1.10.0 (removes warnings) 2. Modified Migrations.sol to use new constructor format 3. Changed keccak256 to explicitly cast to bytes 4. Modified README to start ganache with 7mm gas limit
mahdi-ln · Jun 19, 2018 · 1d0a4e1 · 1d0a4e1
1 parent dd2aa48
commit 1d0a4e1
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -23,7 +23,7 @@ This will display some warnings due to OpenZeppelin library files which can be i
 ## Running Test Network
 
 To run a test ethereum network, in a separate terminal execute:  
-`ganache-cli`
+`ganache-cli --gasLimit 7000000`
 
 ## Deployment
 

diff --git a/contracts/ArbitrationFactory.sol b/contracts/ArbitrationFactory.sol
@@ -45,7 +45,7 @@ contract ArbitrationFactory is Ownable {
    * @notice Returns the hash of an agreement string
    */
   function generateHash(string _input) pure public returns (bytes32) {
-    return keccak256(_input);
+    return keccak256(bytes(_input));
   }
 
 }
diff --git a/contracts/Migrations.sol b/contracts/Migrations.sol
@@ -8,7 +8,7 @@ contract Migrations {
     if (msg.sender == owner) _;
   }
 
-  function Migrations() public {
+  constructor() public {
     owner = msg.sender;
   }
 

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -21,7 +21,7 @@
   "homepage": "https://github.com/jurteam/mvp-smart-contract#readme",
   "dependencies": {
     "ganache-cli": "^6.1.3",
-    "openzeppelin-solidity": "^1.9.0",
+    "openzeppelin-solidity": "^1.10.0",
     "truffle": "^4.1.11"
   },
   "devDependencies": {