Merge branch 'master' into fix_empty_screen_on_array

Author: m4dcoder

README.md

@@ -39,6 +39,12 @@ then you need to bootstrap the micromodules
 $ lerna bootstrap
 ```
 
+to avoid conflicts on node_modules, delete those generated by lerna:
+
+```shell
+$ rm -rf apps/st2-workflows/node_modules/
+```
+
 and finally run build system to fetch the font, compile css and so on
 
 ```shell

apps/st2-actions/actions-details.component.js

@@ -57,6 +57,7 @@ export default class ActionsDetails extends React.Component {
   static propTypes = {
     handleNavigate: PropTypes.func.isRequired,
     handleRun: PropTypes.func.isRequired,
+    handleDelete: PropTypes.func.isRequired,
 
     id: PropTypes.string,
     section: PropTypes.string,
@@ -127,8 +128,6 @@ export default class ActionsDetails extends React.Component {
     }
   }
 
-  
-
   componentDidUpdate(prevProps) {
     const { id } = this.props;
     if (id && id !== prevProps.id) {
@@ -242,6 +241,15 @@ export default class ActionsDetails extends React.Component {
     return this.props.handleRun(...args);
   }
 
+  handleDelete (ref) {
+    const { id } = this.props;
+
+    if (!window.confirm(`You are about to delete the action "${id}". This operation is irreversible. Are you sure?`)) {
+      return undefined;
+    }
+    return this.props.handleDelete(id);
+  }
+
   render() {
     const { section, action, executions, entrypoint } = this.props;
     if (!action) {
@@ -281,6 +289,8 @@ export default class ActionsDetails extends React.Component {
               />
               <Button flat value="Preview" onClick={() => this.handleToggleRunPreview()} />
               <DetailsToolbarSeparator />
+              <Button className="st2-forms__button st2-details__toolbar-button"  value="Delete" onClick={() => this.handleDelete()}  />
+
               { action.runner_type === 'mistral-v2' || action.runner_type === 'orquesta' ? (
                 <Link
                   target="_blank"

apps/st2-actions/actions-panel.component.js

@@ -55,7 +55,6 @@ import './style.css';
   return { collapsed, ...props };
 }, (dispatch, props) => {
   const { uid } = props;
-
   return {
     onToggle: () => store.dispatch(flexActions.toggle(uid)),
   };
@@ -116,7 +115,6 @@ export default class ActionsPanel extends React.Component {
       .then(() => {
         const { id } = this.urlParams;
         const { groups } = this.props;
-
         if (id && groups && !groups.some(({ actions }) => actions.some(({ ref }) => ref === id))) {
           this.navigate({ id: false });
         }
@@ -128,8 +126,7 @@ export default class ActionsPanel extends React.Component {
     const {
       ref = get('groups[0].actions[0].ref', this.props),
       section = 'general',
-    } = this.props.match.params;
-
+    } = this.props.match.params;  
     return {
       id: ref,
       section,
@@ -211,6 +208,30 @@ export default class ActionsPanel extends React.Component {
     });
   }
 
+  handleDelete (ref) {
+    return store.dispatch({
+      type: 'DELETE_ACTION',
+      ref,
+      promise: api.request({
+        method: 'delete',
+        path: `/actions/${ref}`,
+      })
+        .then((res) => {
+          notification.success(`Action "${ref}" has been deleted successfully.`);
+          this.navigate({ id: null });
+          store.dispatch(flexActions.toggleAll());
+          return res;
+        })
+        .catch((err) => {
+          notification.error(`Unable to delete action "${ref}".`, {
+            err,
+            
+          });
+          throw err;
+        }),
+    });
+  }
+
   render() {
     const { groups, filter, collapsed } = this.props;
     const { id, section } = this.urlParams;
@@ -282,6 +303,7 @@ export default class ActionsPanel extends React.Component {
           ref={(ref) => this._details = ref}
           handleNavigate={(...args) => this.navigate(...args)}
           handleRun={(...args) => this.handleRun(...args)}
+          handleDelete={(...arg) => this.handleDelete(...arg)}
 
           id={id}
           section={section}

apps/st2-actions/store.js

@@ -154,6 +154,32 @@ const actionReducer = (state = {}, input) => {
       };
     }
 
+    case 'DELETE_ACTION': {
+      const { ref } = input;
+
+      
+
+      switch(input.status) {
+        case 'success':
+          actions = [ ...actions ]
+            .filter(action => action.ref !== ref)
+          ;
+          groups = makeGroups( actions, filter);
+
+          break;
+        case 'error':
+          break;
+        default:
+          break;
+      }
+
+      return {
+        ...state,
+        actions,
+        groups,
+      };
+    }
+
     case 'SET_FILTER': {
       filter = input.filter;
       groups = makeGroups(actions, filter);

modules/st2-action-reporter/reporters/run-local.js

@@ -28,14 +28,17 @@ export default function runLocal(execution) {
     execution.result && execution.result.stderr ? [
       <div key="error" className={style.source}>Error</div>,
       <Highlight well lines={20} className={style.highlight} key="error-code" code={execution.result.stderr} type="result" id={execution.id} />,
+    ] : execution.result && execution.result.error ? [ 
+      <div key="error" className={style.source}>Error</div>,
+      <Highlight well lines={20} className={style.highlight} key="error-code" code={execution.result.error} type="result" id={execution.id} />,
     ] : null,
 
     execution.result && execution.result.traceback ? [
       <div key="traceback" className={style.source}>Traceback</div>,
       <Highlight well lines={20} className={style.highlight} key="traceback-code" code={[ execution.result.error, execution.result.traceback ].join('\n')} type="result" id={execution.id} />,
     ] : null,
 
-    !execution.result || (!execution.result.stdout && !execution.result.stderr && !execution.result.traceback) ? (
+    !execution.result || (!execution.result.stdout && !execution.result.stderr && !execution.result.error && !execution.result.traceback) ? (
       <Highlight well className={style.highlight} key="none" code="// Action produced no data" type="result" id={execution.id} />
     ) : null,
   ];

modules/st2-action-reporter/reporters/run-python.js

@@ -33,14 +33,17 @@ export default function runPython(execution) {
     execution.result && execution.result.stderr ? [
       <div key="error" className={style.source}>Error</div>,
       <Highlight well lines={20} className={style.highlight} key="error-code" code={execution.result.stderr} type="result" id={execution.id} />,
+    ] : execution.result && execution.result.error ? [ 
+      <div key="error" className={style.source}>Error</div>,
+      <Highlight well lines={20} className={style.highlight} key="error-code" code={execution.result.error} type="result" id={execution.id} />,
     ] : null,
 
     execution.result && execution.result.traceback ? [
       <div key="traceback" className={style.source}>Traceback</div>,
       <Highlight well lines={20} className={style.highlight} key="traceback-code" code={[ execution.result.error, execution.result.traceback ].join('\n')} type="result" id={execution.id} />,
     ] : null,
 
-    !execution.result || (!execution.result.result && !execution.result.stdout && !execution.result.stderr && !execution.result.traceback) ? (
+    !execution.result || (!execution.result.result && !execution.result.stdout && !execution.result.stderr && !execution.result.error && !execution.result.traceback) ? (
       <Highlight well className={style.highlight} key="none" code="// Action produced no data" />
     ) : null,
   ].filter(v => v);

modules/st2-action-reporter/reporters/run-remote.js

@@ -39,14 +39,17 @@ export default function runRemote(execution) {
       result && result.stderr ? [
         <div key="error" className={style.source}>Error</div>,
         <Highlight well lines={20} className={style.highlight} key="error-code" code={result.stderr} type="result" id={execution.id} />,
+      ] :result && result.error ? [
+        <div key="error" className={style.source}>Error</div>,
+        <Highlight well lines={20} className={style.highlight} key="error-code" code={result.error} type="result" id={execution.id} />,
       ] : null,
 
       result && result.traceback ? [
         <div key="traceback" className={style.source}>Traceback</div>,
         <Highlight well lines={20} className={style.highlight} key="traceback-code" code={[ result.error, result.traceback ].join('\n')} type="result" id={execution.id} />,
       ] : null,
 
-      !result || (!result.result && !result.stderr && !result.stdout && !result.traceback) ? (
+      !result || (!result.result && !result.stderr &&  !result.stdout && !result.error  && !result.traceback) ? (
         <Highlight well className={style.highlight} key="none" code="// Action produced no data" />
       ) : null,
     ];