From 67f734cb20110e9932a4ecb8f3df5b8902fe9f3f Mon Sep 17 00:00:00 2001
From: Jannis Redmann <mail@jannisr.de>
Date: Thu, 20 Oct 2016 16:09:58 +0200
Subject: [PATCH] fix linting issues (#2758)

---
 js/src/views/Dapps/AddDapps/AddDapps.js                   | 2 +-
 js/src/views/Dapps/available.js                           | 3 +--
 js/src/views/Signer/components/SignRequest/SignRequest.js | 8 ++++++--
 .../components/TransactionFinished/TransactionFinished.js | 8 ++++++--
 4 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/js/src/views/Dapps/AddDapps/AddDapps.js b/js/src/views/Dapps/AddDapps/AddDapps.js
index 5b327796f..28679a2d5 100644
--- a/js/src/views/Dapps/AddDapps/AddDapps.js
+++ b/js/src/views/Dapps/AddDapps/AddDapps.js
@@ -43,7 +43,7 @@ export default class AddDapps extends Component {
           <Button label={ 'Done' } onClick={ onClose } icon={ <DoneIcon /> } />
         ] }
         visible={ open }
-        scroll={ true }
+        scroll
       >
         <List>
           { available.map(this.renderApp) }
diff --git a/js/src/views/Dapps/available.js b/js/src/views/Dapps/available.js
index ef0590a81..24e1c2c8a 100644
--- a/js/src/views/Dapps/available.js
+++ b/js/src/views/Dapps/available.js
@@ -64,11 +64,10 @@ const hardcoded = [
 ];
 
 export default function () {
-
   // return fetch('//127.0.0.1:8080/api/apps')
   // .then((res) => res.ok ? res.json() : [])
   return Promise.resolve(hardcoded) // TODO
   .then((apps) => apps.map((app) => {
-    return Object.assign({}, app, { hash: sha3(app.id) })
+    return Object.assign({}, app, { hash: sha3(app.id) });
   }));
 }
diff --git a/js/src/views/Signer/components/SignRequest/SignRequest.js b/js/src/views/Signer/components/SignRequest/SignRequest.js
index 55ca82077..2c026edf9 100644
--- a/js/src/views/Signer/components/SignRequest/SignRequest.js
+++ b/js/src/views/Signer/components/SignRequest/SignRequest.js
@@ -22,6 +22,8 @@ import TxHashLink from '../TxHashLink';
 
 import styles from './SignRequest.css';
 
+const nullable = (type) => React.PropTypes.oneOfType([ React.PropTypes.oneOf([ null ]), type ]);
+
 export default class SignRequest extends Component {
 
   // TODO [todr] re-use proptypes?
@@ -34,7 +36,9 @@ export default class SignRequest extends Component {
     onConfirm: PropTypes.func,
     onReject: PropTypes.func,
     status: PropTypes.string,
-    className: PropTypes.string
+    className: PropTypes.string,
+    chain: nullable(PropTypes.object),
+    balance: nullable(PropTypes.object)
   };
 
   state = {
@@ -61,7 +65,7 @@ export default class SignRequest extends Component {
   }
 
   render () {
-    const { chain, balance, className } = this.props;
+    const { className } = this.props;
     return (
       <div className={ `${styles.container} ${className || ''}` }>
         { this.renderDetails() }
diff --git a/js/src/views/Signer/components/TransactionFinished/TransactionFinished.js b/js/src/views/Signer/components/TransactionFinished/TransactionFinished.js
index 080436a37..d7717c7fb 100644
--- a/js/src/views/Signer/components/TransactionFinished/TransactionFinished.js
+++ b/js/src/views/Signer/components/TransactionFinished/TransactionFinished.js
@@ -26,6 +26,8 @@ import styles from './TransactionFinished.css';
 import * as tUtil from '../util/transaction';
 import { capitalize } from '../util/util';
 
+const nullable = (type) => React.PropTypes.oneOfType([ React.PropTypes.oneOf([ null ]), type ]);
+
 export default class TransactionFinished extends Component {
   static contextTypes = {
     api: PropTypes.object.isRequired
@@ -42,7 +44,10 @@ export default class TransactionFinished extends Component {
     to: PropTypes.string, // undefined if it's a contract
     txHash: PropTypes.string, // undefined if transacation is rejected
     className: PropTypes.string,
-    data: PropTypes.string
+    data: PropTypes.string,
+    chain: nullable(PropTypes.object),
+    fromBalance: nullable(PropTypes.object),
+    toBalance: nullable(PropTypes.object)
   };
 
   state = {
@@ -81,7 +86,6 @@ export default class TransactionFinished extends Component {
     }
 
     const { className, date, id } = this.props;
-    const { totalValue } = this.state;
 
     return (
       <div className={ `${styles.container} ${className || ''}` }>