@kineticdata/react 5.1.0 → 5.1.2

package/lib/apis/core/submissions.test.js

@@ -224,7 +224,7 @@ describe('#searchSubmissions', function () {
         }
       };
       search = new SubmissionSearch().build();
-      axios.get = resolvePromiseWith(response);
+      axios.post = resolvePromiseWith(response);
     });
     test('does not return errors', function () {
       expect.assertions(1);
@@ -265,6 +265,33 @@ describe('#searchSubmissions', function () {
       });
     });
   });
+  describe('when successful - get', function () {
+    var response;
+    var search;
+    beforeEach(function () {
+      response = {
+        status: 200,
+        data: {
+          submissions: [],
+          messages: [],
+          nextPageToken: 'page-token'
+        }
+      };
+      search = new SubmissionSearch().build();
+      axios.get = resolvePromiseWith(response);
+    });
+    test('does return submissions', function () {
+      expect.assertions(2);
+      return searchSubmissions({
+        search: search,
+        get: true
+      }).then(function (_ref5) {
+        var submissions = _ref5.submissions;
+        expect(submissions).toBeDefined();
+        expect(submissions).toBeInstanceOf(Array);
+      });
+    });
+  });
 });
 describe('#fetchSubmission', function () {
   describe('when successful', function () {
@@ -287,8 +314,8 @@ describe('#fetchSubmission', function () {
       expect.assertions(1);
       return fetchSubmission({
         id: id
-      }).then(function (_ref5) {
-        var errors = _ref5.errors;
+      }).then(function (_ref6) {
+        var errors = _ref6.errors;
         expect(errors).toBeUndefined();
       });
     });
@@ -296,8 +323,8 @@ describe('#fetchSubmission', function () {
       expect.assertions(1);
       return fetchSubmission({
         id: id
-      }).then(function (_ref6) {
-        var submission = _ref6.submission;
+      }).then(function (_ref7) {
+        var submission = _ref7.submission;
         expect(submission).toMatchObject({
           id: id,
           values: values
@@ -323,8 +350,8 @@ describe('#fetchSubmission', function () {
       expect.assertions(1);
       return fetchSubmission({
         id: 'fake'
-      }).then(function (_ref7) {
-        var error = _ref7.error;
+      }).then(function (_ref8) {
+        var error = _ref8.error;
         expect(error).toEqual({
           statusCode: 500,
           key: null,
@@ -359,8 +386,8 @@ describe('#createSubmission', function () {
         kappSlug: kappSlug,
         formSlug: formSlug,
         values: values
-      }).then(function (_ref8) {
-        var errors = _ref8.errors;
+      }).then(function (_ref9) {
+        var errors = _ref9.errors;
         expect(errors).toBeUndefined();
       });
     });
@@ -370,8 +397,8 @@ describe('#createSubmission', function () {
         kappSlug: kappSlug,
         formSlug: formSlug,
         values: values
-      }).then(function (_ref9) {
-        var submission = _ref9.submission;
+      }).then(function (_ref10) {
+        var submission = _ref10.submission;
         expect(submission).toMatchObject({
           id: id,
           values: values
@@ -443,8 +470,8 @@ describe('#createSubmission', function () {
       return createSubmission({
         formSlug: formSlug,
         values: values
-      }).then(function (_ref10) {
-        var error = _ref10.error;
+      }).then(function (_ref11) {
+        var error = _ref11.error;
         expect(error).toEqual({
           statusCode: 500,
           key: null,
@@ -475,8 +502,8 @@ describe('#deleteSubmission', function () {
       expect.assertions(1);
       return deleteSubmission({
         id: id
-      }).then(function (_ref11) {
-        var errors = _ref11.errors;
+      }).then(function (_ref12) {
+        var errors = _ref12.errors;
         expect(errors).toBeUndefined();
       });
     });
@@ -484,8 +511,8 @@ describe('#deleteSubmission', function () {
       expect.assertions(1);
       return deleteSubmission({
         id: id
-      }).then(function (_ref12) {
-        var submission = _ref12.submission;
+      }).then(function (_ref13) {
+        var submission = _ref13.submission;
         expect(submission).toMatchObject({
           id: id,
           values: values
@@ -511,8 +538,8 @@ describe('#deleteSubmission', function () {
       expect.assertions(1);
       return deleteSubmission({
         id: 'fake'
-      }).then(function (_ref13) {
-        var error = _ref13.error;
+      }).then(function (_ref14) {
+        var error = _ref14.error;
         expect(error).toEqual({
           statusCode: 500,
           key: null,

package/lib/apis/core/webhooks.js

@@ -24,7 +24,7 @@ var buildEndpoint = function buildEndpoint(_ref) {
   var kappSlug = _ref.kappSlug,
       webhookName = _ref.webhookName;
   var basePath = kappSlug ? "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/webhooks") : "".concat(bundle.apiLocation(), "/webhooks");
-  return webhookName ? "".concat(basePath, "/").concat(webhookName) : basePath;
+  return webhookName ? "".concat(basePath, "/").concat(encodeURIComponent(webhookName)) : basePath;
 };
 
 export var fetchWebhooks = function fetchWebhooks() {

package/lib/apis/core/webhooks.test.js

@@ -185,7 +185,7 @@ describe('webhooks api', function () {
 
             case 3:
               result = _context4.sent;
-              expect(axios.get.mock.calls).toEqual([['space/app/api/v1/webhooks/Test Webhook Name', {
+              expect(axios.get.mock.calls).toEqual([['space/app/api/v1/webhooks/Test%20Webhook%20Name', {
                 params: {},
                 headers: {
                   'X-Kinetic-AuthAssumed': 'true'
@@ -391,7 +391,7 @@ describe('webhooks api', function () {
 
             case 3:
               result = _context7.sent;
-              expect(axios.put.mock.calls).toEqual([['space/app/api/v1/webhooks/Test Webhook Name', {
+              expect(axios.put.mock.calls).toEqual([['space/app/api/v1/webhooks/Test%20Webhook%20Name', {
                 authStrategy: null,
                 event: 'Login Failure',
                 filter: '',

package/lib/apis/core/workflows.js

@@ -0,0 +1,56 @@
+import { apiGroup, handleErrors, headerBuilder, paramBuilder } from '../http';
+import axios from 'axios';
+import { bundle } from '../../helpers';
+
+var _apiGroup = apiGroup({
+  name: 'Workflow',
+  dataOption: 'workflow',
+  plural: {
+    requiredOptions: [],
+    url: function url(_ref) {
+      var kappSlug = _ref.kappSlug,
+          formSlug = _ref.formSlug;
+      return formSlug ? "/kapps/".concat(kappSlug, "/forms/").concat(formSlug, "/workflows") : kappSlug ? "/kapps/".concat(kappSlug, "/workflows") : "/workflows";
+    },
+    transform: function transform(response) {
+      return response.data;
+    }
+  },
+  singular: {
+    requiredOptions: ['workflowId'],
+    url: function url(_ref2) {
+      var workflowId = _ref2.workflowId,
+          kappSlug = _ref2.kappSlug,
+          formSlug = _ref2.formSlug;
+      return formSlug ? "/kapps/".concat(kappSlug, "/forms/").concat(formSlug, "/workflows/").concat(workflowId) : kappSlug ? "/kapps/".concat(kappSlug, "/workflows/").concat(workflowId) : "/workflows/".concat(workflowId);
+    },
+    transform: function transform(response) {
+      return {
+        workflow: response.data
+      };
+    }
+  }
+}),
+    fetchWorkflows = _apiGroup.fetchWorkflows,
+    fetchWorkflow = _apiGroup.fetchWorkflow,
+    createWorkflow = _apiGroup.createWorkflow,
+    updateWorkflow = _apiGroup.updateWorkflow,
+    deleteWorkflow = _apiGroup.deleteWorkflow;
+
+export { fetchWorkflows, fetchWorkflow, createWorkflow, updateWorkflow, deleteWorkflow };
+
+var repairPath = function repairPath(_ref3) {
+  var kappSlug = _ref3.kappSlug,
+      formSlug = _ref3.formSlug;
+  return kappSlug && formSlug ? "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/forms/").concat(formSlug, "/workflows/repair") : kappSlug ? "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/workflows/repair") : "".concat(bundle.apiLocation(), "/workflows/repair");
+};
+
+export var repairWorkflows = function repairWorkflows() {
+  var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+  return axios.post(repairPath(options), {
+    params: paramBuilder(options),
+    headers: headerBuilder(options)
+  }).then(function (response) {
+    return response.data;
+  })["catch"](handleErrors);
+};

package/lib/apis/http.js

@@ -17,6 +17,12 @@ export var handleErrors = function handleErrors(error) {
   // make a mistake in a `then` block in one of our api functions.
   if (error instanceof Error && !error.response) {
     throw error;
+  }
+
+  if (axios.isCancel(error)) {
+    return {
+      error: 'Canceled by user request.'
+    };
   } // Destructure out the information needed.
 
 

package/lib/apis/index.js

@@ -22,6 +22,7 @@ export * from './core/meta';
 export * from './core/notices';
 export * from './core/oauthClients';
 export * from './core/platformComponents';
+export * from './core/platformItems';
 export * from './core/profile';
 export * from './core/securityPolicyDefinitions';
 export * from './core/space';
@@ -32,7 +33,8 @@ export * from './core/users';
 export * from './core/version';
 export * from './core/webApis';
 export * from './core/webhooks';
-export * from './core/webhooksJobs'; // Discussions
+export * from './core/webhooksJobs';
+export * from './core/workflows'; // Discussions
 
 export * from './discussions'; // Http
 

package/lib/apis/task/index.js

@@ -42,9 +42,12 @@ export var fetchTrees = function fetchTrees() {
 };
 export var fetchTree = function fetchTree() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-  validateOptions('fetchTree', ['name'], options);
-  var id = buildTreeId(options);
-  return axios.get("".concat(bundle.spaceLocation(), "/app/components/task/app/api/v2/trees/").concat(id), {
+
+  if (!options.guid) {
+    validateOptions('fetchTree', ['name'], options);
+  }
+
+  return axios.get(options.guid ? "".concat(bundle.spaceLocation(), "/app/components/task/app/api/v2/trees/guid/").concat(options.guid) : "".concat(bundle.spaceLocation(), "/app/components/task/app/api/v2/trees/").concat(buildTreeId(options)), {
     params: {
       include: options.include
     }
@@ -635,6 +638,7 @@ export var fetchTaskRuns = function fetchTaskRuns() {
     return {
       runs: response.data.runs,
       count: response.data.count,
+      more: response.data.more,
       nextPageToken: generateNextPageToken(response.data)
     };
   })["catch"](handleErrors);

package/lib/components/index.js

@@ -119,6 +119,8 @@ export { TriggerTable } from './task/triggers/TriggerTable';
 export { RunErrorTable } from './task/errors/RunErrorTable';
 export { SystemErrorsTable } from './task/errors/SystemErrorsTable';
 export { WorkflowTable } from './task/workflows/WorkflowTable';
+export { LinkedWorkflowTable } from './task/workflows/LinkedWorkflowTable';
+export { LinkedWorkflowForm } from './task/workflows/LinkedWorkflowForm';
 export { MissingRoutineTable } from './task/workflows/MissingRoutineTable';
 export { WorkflowForm } from './task/workflows/WorkflowForm';
 export { SourceTable } from './task/sources/SourceTable';

package/lib/components/table/Table.js

@@ -38,7 +38,8 @@ var TableComponent = function TableComponent(props) {
         components = props.components,
         tableKey = props.tableKey,
         _filterFormKey = props.filterFormKey,
-        count = props.count;
+        count = props.count,
+        extraData = props.extraData;
     var table = buildTable(props);
     var filter = components.FilterForm ? buildFilterForm(props) : buildFilterLayout(props);
     var pagination = buildPaginationControl(props);
@@ -53,7 +54,8 @@ var TableComponent = function TableComponent(props) {
       loading: loading,
       rows: rows,
       count: count,
-      error: error
+      error: error,
+      extraData: extraData
     });
   }
 
@@ -266,7 +268,8 @@ var buildPaginationControl = function buildPaginationControl(props) {
       count = props.count,
       components = props.components,
       loading = props.loading,
-      renderOptions = props.renderOptions;
+      renderOptions = props.renderOptions,
+      extraData = props.extraData;
   var PaginationControl = components.PaginationControl;
   var prevPage = hasPrevPage(data, dataSource, tableOptions, pageTokens, pageOffset) ? onPrevPage(tableKey) : null;
   var nextPage = hasNextPage(data, dataSource, tableOptions, pageOffset, pageSize, currentPageToken, rows) ? onNextPage(tableKey) : null;
@@ -286,7 +289,8 @@ var buildPaginationControl = function buildPaginationControl(props) {
     endIndex: endIndex,
     count: count,
     renderOptions: renderOptions,
-    tableOptions: tableOptions
+    tableOptions: tableOptions,
+    extraData: extraData
   });
 };
 
@@ -306,7 +310,8 @@ export var buildTable = function buildTable(props) {
     error: props.error,
     empty: props.rows.isEmpty(),
     renderOptions: props.renderOptions,
-    tableOptions: props.tableOptions
+    tableOptions: props.tableOptions,
+    extraData: props.extraData
   });
 };
 export var buildTableHeader = function buildTableHeader(props) {
@@ -412,7 +417,8 @@ export var buildTableBodyCells = function buildTableBodyCells(props, row, rowInd
       columnComponents = props.columnComponents,
       appliedFilters = props.appliedFilters,
       renderOptions = props.renderOptions,
-      tableOptions = props.tableOptions;
+      tableOptions = props.tableOptions,
+      extraData = props.extraData;
   return fromColumnSet(columns, columnSet).map(function (column, index) {
     var BodyCell = columnComponents.getIn([column.get('value'), 'BodyCell'], components.BodyCell);
     var value = row.get(column.get('value'));
@@ -428,7 +434,8 @@ export var buildTableBodyCells = function buildTableBodyCells(props, row, rowInd
       column: column,
       filters: appliedFilters,
       renderOptions: renderOptions,
-      tableOptions: tableOptions
+      tableOptions: tableOptions,
+      extraData: extraData
     }));
   });
 };

package/lib/components/table/Table.redux.js

@@ -138,6 +138,7 @@ regHandlers({
         onFetch = _ref3$payload.onFetch;
     return !state.getIn(['tables', tableKey, 'mounted']) ? state : state.hasIn(['tables', tableKey, 'configured']) ? state.setIn(['tables', tableKey, 'initialize'], false) : state.mergeIn(['tables', tableKey], Map({
       data: hasData(data) ? fromJS(data) : data,
+      extraData: null,
       dataSource: dataSource,
       tableOptions: tableOptions,
       columns: columns,
@@ -170,12 +171,14 @@ regHandlers({
         tableKey = _ref4$payload.tableKey,
         rows = _ref4$payload.rows,
         data = _ref4$payload.data,
+        _ref4$payload$extraDa = _ref4$payload.extraData,
+        extraData = _ref4$payload$extraDa === void 0 ? null : _ref4$payload$extraDa,
         nextPageToken = _ref4$payload.nextPageToken,
         count = _ref4$payload.count,
         _ref4$payload$error = _ref4$payload.error,
         error = _ref4$payload$error === void 0 ? null : _ref4$payload$error;
     return state.updateIn(['tables', tableKey], function (table) {
-      return table && table.set('rows', rows).set('data', data).set('currentPageToken', nextPageToken).set('nextPageToken', null).set('count', count).set('error', error).set('initializing', false).set('loading', false);
+      return table && table.set('rows', rows).set('data', data).set('extraData', extraData).set('currentPageToken', nextPageToken).set('nextPageToken', null).set('count', count).set('error', error).set('initializing', false).set('loading', false);
     });
   },
   NEXT_PAGE: function NEXT_PAGE(state, _ref5) {
@@ -250,7 +253,7 @@ regHandlers({
 });
 
 function calculateRowsTask(_ref15) {
-  var payload, tableKey, tableData, response, rows, data, nextPageToken, count, error, onFetch;
+  var payload, tableKey, tableData, response, rows, data, nextPageToken, count, error, extraData, onFetch;
   return _regeneratorRuntime.wrap(function calculateRowsTask$(_context) {
     while (1) {
       switch (_context.prev = _context.next) {
@@ -279,7 +282,7 @@ function calculateRowsTask(_ref15) {
 
         case 10:
           response = _context.sent;
-          rows = response.rows, data = response.data, nextPageToken = response.nextPageToken, count = response.count, error = response.error;
+          rows = response.rows, data = response.data, nextPageToken = response.nextPageToken, count = response.count, error = response.error, extraData = response.extraData;
           onFetch = tableData.get('onFetch');
 
           if (!error) {
@@ -296,7 +299,8 @@ function calculateRowsTask(_ref15) {
               rows: List(),
               data: List(),
               nextPageToken: null,
-              count: null
+              count: null,
+              extraData: extraData
             }
           });
 
@@ -313,7 +317,8 @@ function calculateRowsTask(_ref15) {
               rows: rows,
               data: data,
               nextPageToken: nextPageToken,
-              count: count
+              count: count,
+              extraData: extraData
             }
           });
 
@@ -328,7 +333,8 @@ function calculateRowsTask(_ref15) {
             tableKey: tableKey,
             rows: rows,
             count: count,
-            error: error
+            error: error,
+            extraData: extraData
           });
 
         case 23:
@@ -647,7 +653,8 @@ var calculateRows = function calculateRows(tableData) {
       var _transform = transform(response, paramData),
           nextPageToken = _transform.nextPageToken,
           responseData = _transform.data,
-          count = _transform.count;
+          count = _transform.count,
+          extraData = _transform.extraData;
 
       var data = fromJS(responseData);
       var transformedRows = transformData(data, tableData);
@@ -656,7 +663,8 @@ var calculateRows = function calculateRows(tableData) {
         nextPageToken: nextPageToken,
         count: dataSource.clientSideSearch || dataSource.clientSide ? rows.size : count,
         data: data,
-        rows: rows
+        rows: rows,
+        extraData: extraData
       };
     });
   } else {

package/lib/components/task/builder/TaskDefinitionConfigForm.js

@@ -11,7 +11,7 @@ export var checkOmittedParameters = function checkOmittedParameters(node, parame
   }
 };
 export var generateTaskDefinition = function generateTaskDefinition(taskDefinition) {
-  if (taskDefinition.definitionName === 'system_submission_create') {
+  if (taskDefinition && taskDefinition.definitionName === 'system_submission_create') {
     return function (_ref) {
       var form = _ref.form;
       return _objectSpread(_objectSpread({}, taskDefinition), {}, {

package/lib/components/task/builder/builder.redux.js

@@ -7,7 +7,7 @@ import { List, OrderedMap } from 'immutable';
 import { isFunction } from 'lodash-es';
 import { action, dispatch, regHandlers, regSaga } from '../../../store';
 import { deserializeTree, serializeTree, Connector, TreeBuilderState, deserializeWebApi } from './models';
-import { cloneTree, fetchTaskCategories, fetchTree, fetchWebApi, updateTree, updateWebApi } from '../../../apis';
+import { createWorkflow, createTree, fetchPlatformItem, fetchTaskCategories, fetchTree, fetchWebApi, updateTree, updateWebApi } from '../../../apis';
 import { renameDependencies, treeReturnTask } from './helpers';
 export var mountTreeBuilder = function mountTreeBuilder(treeKey) {
   return dispatch('TREE_MOUNT', {
@@ -34,7 +34,7 @@ var remember = function remember(state, treeKey) {
 };
 
 regSaga(takeEvery('TREE_CONFIGURE', /*#__PURE__*/_regeneratorRuntime.mark(function _callee(_ref) {
-  var payload, name, sourceGroup, sourceName, treeKey, webApiProps, _yield$all, _yield$all2, tree, categories, webApi;
+  var payload, name, sourceGroup, sourceName, treeKey, webApiProps, _yield$all, _yield$all2, tree, categories, webApi, platformItem, result;
 
   return _regeneratorRuntime.wrap(function _callee$(_context) {
     while (1) {
@@ -62,33 +62,62 @@ regSaga(takeEvery('TREE_CONFIGURE', /*#__PURE__*/_regeneratorRuntime.mark(functi
           tree = _yield$all2[0].tree;
           categories = _yield$all2[1].categories;
           webApi = _yield$all2[2].webApi;
-          _context.next = 13;
+          platformItem = null;
+
+          if (!tree.event) {
+            _context.next = 17;
+            break;
+          }
+
+          _context.next = 15;
+          return call(fetchPlatformItem, {
+            type: tree.platformItemType,
+            id: tree.platformItemId
+          });
+
+        case 15:
+          result = _context.sent;
+          platformItem = result.platformItem;
+
+        case 17:
+          _context.next = 19;
           return put(action('TREE_LOADED', {
             categories: categories,
             kappSlug: webApiProps && webApiProps.kappSlug,
+            platformItem: platformItem,
             treeKey: treeKey,
             tree: deserializeTree(tree),
             webApi: webApiProps && deserializeWebApi(webApi, webApiProps.kappSlug)
           }));
 
-        case 13:
-          _context.next = 18;
+        case 19:
+          _context.next = 24;
           break;
 
-        case 15:
-          _context.prev = 15;
+        case 21:
+          _context.prev = 21;
           _context.t0 = _context["catch"](1);
           console.error('Caught error loading tree', _context.t0);
 
-        case 18:
+        case 24:
         case "end":
           return _context.stop();
       }
     }
-  }, _callee, null, [[1, 15]]);
+  }, _callee, null, [[1, 21]]);
 })));
+
+var getPlatformItemSlugs = function getPlatformItemSlugs(platformItem) {
+  return platformItem.kapp ? {
+    formSlug: platformItem.slug,
+    kappSlug: platformItem.kapp.slug
+  } : platformItem.space ? {
+    kappSlug: platformItem.slug
+  } : {};
+};
+
 regSaga(takeEvery('TREE_SAVE', /*#__PURE__*/_regeneratorRuntime.mark(function _callee2(_ref2) {
-  var payload, newName, onError, onSave, overwrite, treeKey, _yield$select, kappSlug, lastSave, lastWebApi, tree, webApi, name, sourceGroup, sourceName, _yield, error1, newTree, _yield2, error2, error;
+  var payload, newName, onError, onSave, overwrite, treeKey, _yield$select, kappSlug, lastSave, lastWebApi, platformItem, tree, webApi, name, sourceGroup, sourceName, _yield, error1, newTree, newWorkflow, _yield2, error2, error;
 
   return _regeneratorRuntime.wrap(function _callee2$(_context2) {
     while (1) {
@@ -109,19 +138,23 @@ regSaga(takeEvery('TREE_SAVE', /*#__PURE__*/_regeneratorRuntime.mark(function _c
           kappSlug = _yield$select.kappSlug;
           lastSave = _yield$select.lastSave;
           lastWebApi = _yield$select.lastWebApi;
+          platformItem = _yield$select.platformItem;
           tree = _yield$select.tree;
           webApi = _yield$select.webApi;
           name = lastSave.name, sourceGroup = lastSave.sourceGroup, sourceName = lastSave.sourceName; // if a newName was passed we will be creating a new tree with the builder
           // contents, otherwise just an update
+          // additionally, if the tree is a linked workflow then we call a core
+          // endpoint to create the workflow
 
-          _context2.next = 14;
-          return newName ? call(cloneTree, {
-            name: name,
-            sourceGroup: sourceGroup,
-            sourceName: sourceName,
-            tree: _objectSpread({
+          _context2.next = 15;
+          return newName ? tree.event ? call(createWorkflow, _objectSpread({
+            workflow: _objectSpread(_objectSpread({}, serializeTree(tree)), {}, {
               name: newName
-            }, serializeTree(tree, true))
+            })
+          }, getPlatformItemSlugs(platformItem))) : call(createTree, {
+            tree: _objectSpread(_objectSpread({}, serializeTree(tree, true)), {}, {
+              name: newName
+            })
           }) : call(updateTree, {
             name: name,
             sourceGroup: sourceGroup,
@@ -129,23 +162,24 @@ regSaga(takeEvery('TREE_SAVE', /*#__PURE__*/_regeneratorRuntime.mark(function _c
             tree: serializeTree(tree, overwrite)
           });
 
-        case 14:
+        case 15:
           _yield = _context2.sent;
           error1 = _yield.error;
           newTree = _yield.tree;
-          _context2.next = 19;
+          newWorkflow = _yield.workflow;
+          _context2.next = 21;
           return webApi && !error1 ? call(updateWebApi, {
             slug: lastWebApi.get('slug'),
             kappSlug: kappSlug,
             webApi: webApi
           }) : {};
 
-        case 19:
+        case 21:
           _yield2 = _context2.sent;
           error2 = _yield2.error;
           error = error1 || error2; // dispatch the appropriate action based on the result of the call above
 
-          _context2.next = 24;
+          _context2.next = 26;
           return put(error ? action('TREE_SAVE_ERROR', {
             treeKey: treeKey,
             error: error.message || error,
@@ -153,26 +187,26 @@ regSaga(takeEvery('TREE_SAVE', /*#__PURE__*/_regeneratorRuntime.mark(function _c
           }) : action('TREE_SAVE_SUCCESS', {
             previousTree: lastSave,
             treeKey: treeKey,
-            tree: newTree,
+            tree: newTree || newWorkflow,
             webApi: webApi,
             onSave: onSave
           }));
 
-        case 24:
-          _context2.next = 29;
+        case 26:
+          _context2.next = 31;
           break;
 
-        case 26:
-          _context2.prev = 26;
+        case 28:
+          _context2.prev = 28;
           _context2.t0 = _context2["catch"](1);
           console.error(_context2.t0);
 
-        case 29:
+        case 31:
         case "end":
           return _context2.stop();
       }
     }
-  }, _callee2, null, [[1, 26]]);
+  }, _callee2, null, [[1, 28]]);
 })));
 regSaga(takeEvery('TREE_SAVE_ERROR', /*#__PURE__*/_regeneratorRuntime.mark(function _callee3(_ref3) {
   var _ref3$payload, error, onError;
@@ -269,6 +303,7 @@ regHandlers({
     var _ref8$payload = _ref8.payload,
         categories = _ref8$payload.categories,
         kappSlug = _ref8$payload.kappSlug,
+        platformItem = _ref8$payload.platformItem,
         treeKey = _ref8$payload.treeKey,
         tree = _ref8$payload.tree,
         webApi = _ref8$payload.webApi;
@@ -277,6 +312,7 @@ regHandlers({
       lastSave: tree,
       lastWebApi: webApi,
       loading: false,
+      platformItem: platformItem,
       tasks: List(categories).map(function (category) {
         return category.name === 'System Controls' ? _objectSpread(_objectSpread({}, category), {}, {
           handlers: [].concat(_toConsumableArray(category.handlers), [treeReturnTask(tree)])
@@ -314,6 +350,7 @@ regHandlers({
         webApi = _ref11$payload.webApi;
     var newTree = state.getIn(['trees', treeKey, 'tree']).merge({
       name: tree.name,
+      sourceGroup: tree.sourceGroup,
       versionId: tree.versionId
     });
     return state.mergeIn(['trees', treeKey], {