Feat/external txs

src/aragonjs-wrapper.js

@@ -467,9 +467,7 @@ const templateParamFilters = {
 
     if (neededSignatures < 1 || neededSignatures > signers.length) {
       throw new Error(
-        `neededSignatures must be between 1 and the total number of signers (${
-          signers.length
-        })`,
+        `neededSignatures must be between 1 and the total number of signers (${signers.length})`,
         neededSignatures
       )
     }

src/components/SignerPanel/ActionPathsContent.js

@@ -12,6 +12,7 @@ const RADIO_ITEM_TITLE_LENGTH = 30
 class ActionPathsContent extends React.Component {
   static propTypes = {
     direct: PropTypes.bool.isRequired,
+    external: PropTypes.bool,
     intent: PropTypes.object.isRequired,
     dao: PropTypes.string.isRequired,
     onSign: PropTypes.func.isRequired,
@@ -175,6 +176,7 @@ class ActionPathsContent extends React.Component {
     const {
       intent,
       direct,
+      external,
       paths,
       pretransaction,
       signingEnabled,
@@ -222,6 +224,14 @@ class ActionPathsContent extends React.Component {
             </Info.Action>
           </div>
         )}
+        {external && (
+          <div css="margin-top: 20px">
+            <Info.Alert title="Warning">
+              This is an attempt to execute a transaction on an external
+              contract. Please take necessary security precautions.
+            </Info.Alert>
+          </div>
+        )}
         <SignerButton onClick={this.handleSign} disabled={!signingEnabled}>
           Create transaction
         </SignerButton>

src/components/SignerPanel/ConfirmTransaction.js

@@ -6,6 +6,7 @@ import ImpossibleContent from './ImpossibleContent'
 const ConfirmTransaction = ({
   direct,
   intent,
+  external,
   dao,
   onClose,
   onSign,
@@ -20,9 +21,10 @@ const ConfirmTransaction = ({
 
   return possible ? (
     <ActionPathsContent
-      intent={intent}
       direct={direct}
       dao={dao}
+      external={external}
+      intent={intent}
       onSign={onSign}
       paths={paths}
       pretransaction={pretransaction}
@@ -37,6 +39,7 @@ const ConfirmTransaction = ({
 ConfirmTransaction.propTypes = {
   direct: PropTypes.bool.isRequired,
   intent: PropTypes.object.isRequired,
+  external: PropTypes.bool,
   dao: PropTypes.string,
   onClose: PropTypes.func.isRequired,
   onSign: PropTypes.func.isRequired,

src/components/SignerPanel/SignerPanel.js

@@ -115,6 +115,7 @@ class SignerPanel extends React.PureComponent {
 
     return {
       intent: (transaction && this.transactionIntent(bag)) || {},
+      external: transaction.external,
       directPath: decoratedPaths.length === 1,
       actionPaths: decoratedPaths.length ? [decoratedPaths] : [],
       pretransaction: (transaction && transaction.pretransaction) || null,
@@ -297,6 +298,7 @@ class SignerPanel extends React.PureComponent {
       actionPaths,
       pretransaction,
       status,
+      external,
     } = this.state
 
     const isTransaction = isTxSignerStatus(status)
@@ -344,6 +346,7 @@ class SignerPanel extends React.PureComponent {
                             <ConfirmTransaction
                               dao={dao}
                               direct={directPath}
+                              external={external}
                               intent={intent}
                               onClose={this.handleSignerClose}
                               onSign={this.handleSign}