Transfer only what's necessary for staking for the storage provider

role.
mnaamani · Apr 9, 2019 · c0f76a0 · c0f76a0
1 parent 341226b
commit c0f76a0
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/bin/cli.js b/bin/cli.js
@@ -255,7 +255,7 @@ async function run_signup(account_file)
   console.log('Identity stored in', filename);
 
   // Ok, transfer for staking.
-  await api.transferForStaking(member_address, role_address);
+  await api.transferForStaking(member_address, role_address, ROLE_STORAGE);
   console.log('Funds transferred.');
 
   // Now apply for the role

diff --git a/lib/joystream/substrate/balances.js b/lib/joystream/substrate/balances.js
@@ -51,11 +51,6 @@ class BalancesApi extends IdentitiesApi
     return await this.api.tx.balances.transfer(to_decoded, amount)
       .signAndSend(from_key);
   }
-
-  async transferForStaking(from, to)
-  {
-    return await this.transfer(from, to, DEFAULT_STAKING_AMOUNT + 100 /*FIXME */);
-  }
 }
 
 module.exports = {

diff --git a/lib/joystream/substrate/roles.js b/lib/joystream/substrate/roles.js
@@ -63,6 +63,13 @@ class RolesApi extends BalancesApi
     return await this.hasMinimumBalanceOf(accountId, required);
   }
 
+  async transferForStaking(from, to, role)
+  {
+    const required = await this.requiredBalanceForRoleStaking(role);
+    return await this.transfer(from, to, required);
+  }
+
+
   async applyForRole(roleAccountId, role, memberAccountId)
   {
     const memberId = await this.memberIdOf(memberAccountId);