@atlaskit/editor-synced-block-provider 2.12.1 → 2.12.2

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # @atlaskit/editor-synced-block-provider
 
+## 2.12.2
+
+### Patch Changes
+
+- [`fd932d0fad4a5`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/fd932d0fad4a5) -
+  [ux] EDITOR-3693 separate sync block creation and update into different provider calls
+- [`1f90e6ac4a228`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/1f90e6ac4a228) -
+  [ux] EDITOR-3662 fix sync block error merging logic to retain old information on error
+
 ## 2.12.1
 
 ### Patch Changes

package/dist/cjs/providers/block-service/blockServiceAPI.js

@@ -210,88 +210,121 @@ var BlockServiceADFWriteProvider = /*#__PURE__*/function () {
               _context3.prev = 7;
               _context3.t0 = _context3["catch"](1);
               if (!(_context3.t0 instanceof _blockService.BlockError)) {
-                _context3.next = 16;
+                _context3.next = 11;
                 break;
               }
-              if (!(_context3.t0.status === 404)) {
-                _context3.next = 15;
-                break;
-              }
-              _context3.next = 13;
-              return (0, _blockService.createSyncedBlock)({
-                blockAri: resourceId,
-                blockInstanceId: data.blockInstanceId,
-                sourceAri: this.sourceAri,
-                product: this.product,
-                content: JSON.stringify(data.content)
-              });
-            case 13:
-              _context3.next = 16;
-              break;
-            case 15:
               return _context3.abrupt("return", {
                 error: mapBlockError(_context3.t0),
                 resourceId: resourceId
               });
-            case 16:
+            case 11:
               return _context3.abrupt("return", {
                 error: (0, _errorHandling.stringifyError)(_context3.t0),
                 resourceId: resourceId
               });
-            case 17:
+            case 12:
             case "end":
               return _context3.stop();
           }
-        }, _callee3, this, [[1, 7]]);
+        }, _callee3, null, [[1, 7]]);
       }));
       function writeData(_x3) {
         return _writeData.apply(this, arguments);
       }
       return writeData;
-    }() // soft deletes the source synced block
+    }()
   }, {
-    key: "deleteData",
+    key: "createData",
     value: function () {
-      var _deleteData = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee4(resourceId) {
+      var _createData = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee4(data) {
+        var resourceId;
         return _regenerator.default.wrap(function _callee4$(_context4) {
           while (1) switch (_context4.prev = _context4.next) {
             case 0:
-              _context4.prev = 0;
-              _context4.next = 3;
+              resourceId = data.resourceId;
+              _context4.prev = 1;
+              _context4.next = 4;
+              return (0, _blockService.createSyncedBlock)({
+                blockAri: resourceId,
+                blockInstanceId: data.blockInstanceId,
+                sourceAri: this.sourceAri,
+                product: this.product,
+                content: JSON.stringify(data.content)
+              });
+            case 4:
+              return _context4.abrupt("return", {
+                resourceId: resourceId
+              });
+            case 7:
+              _context4.prev = 7;
+              _context4.t0 = _context4["catch"](1);
+              if (!(_context4.t0 instanceof _blockService.BlockError)) {
+                _context4.next = 11;
+                break;
+              }
+              return _context4.abrupt("return", {
+                error: mapBlockError(_context4.t0),
+                resourceId: resourceId
+              });
+            case 11:
+              return _context4.abrupt("return", {
+                error: (0, _errorHandling.stringifyError)(_context4.t0),
+                resourceId: resourceId
+              });
+            case 12:
+            case "end":
+              return _context4.stop();
+          }
+        }, _callee4, this, [[1, 7]]);
+      }));
+      function createData(_x4) {
+        return _createData.apply(this, arguments);
+      }
+      return createData;
+    }() // soft deletes the source synced block
+  }, {
+    key: "deleteData",
+    value: function () {
+      var _deleteData = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee5(resourceId) {
+        return _regenerator.default.wrap(function _callee5$(_context5) {
+          while (1) switch (_context5.prev = _context5.next) {
+            case 0:
+              _context5.prev = 0;
+              _context5.next = 3;
               return (0, _blockService.deleteSyncedBlock)({
                 blockAri: resourceId
               });
             case 3:
-              return _context4.abrupt("return", {
+              return _context5.abrupt("return", {
                 resourceId: resourceId,
                 success: true,
                 error: undefined
               });
             case 6:
-              _context4.prev = 6;
-              _context4.t0 = _context4["catch"](0);
-              if (!(_context4.t0 instanceof _blockService.BlockError)) {
-                _context4.next = 10;
+              _context5.prev = 6;
+              _context5.t0 = _context5["catch"](0);
+              if (!(_context5.t0 instanceof _blockService.BlockError)) {
+                _context5.next = 10;
                 break;
               }
-              return _context4.abrupt("return", {
+              return _context5.abrupt("return", {
                 resourceId: resourceId,
                 success: false,
-                error: mapBlockError(_context4.t0)
+                error: mapBlockError(_context5.t0)
               });
             case 10:
-              return _context4.abrupt("return", {
+              return _context5.abrupt("return", {
                 resourceId: resourceId,
                 success: false,
-                error: (0, _errorHandling.stringifyError)(_context4.t0)
+                error: (0, _errorHandling.stringifyError)(_context5.t0)
               });
             case 11:
             case "end":
-              return _context4.stop();
+              return _context5.stop();
           }
-        }, _callee4, null, [[0, 6]]);
+        }, _callee5, null, [[0, 6]]);
       }));
-      function deleteData(_x4) {
+      function deleteData(_x5) {
         return _deleteData.apply(this, arguments);
       }
       return deleteData;

package/dist/cjs/providers/confluence/confluenceContentAPI.js

@@ -260,37 +260,23 @@ var ConfluenceADFWriteProvider = /*#__PURE__*/function () {
                 resourceId: resourceId
               });
             case 23:
-              if (updateResult) {
-                _context3.next = 27;
-                break;
-              }
-              return _context3.abrupt("return", this.createNewContentProperty(pageId, key, syncBlockDataWithSourceDocumentAri, pageType).then(function () {
-                return {
-                  resourceId: resourceId
-                };
-              }, function (error) {
-                return {
-                  error: error
-                };
-              }));
-            case 27:
               return _context3.abrupt("return", {
                 error: "Failed to update ".concat(pageType, " content property")
               });
-            case 28:
-              _context3.next = 33;
+            case 24:
+              _context3.next = 29;
               break;
-            case 30:
-              _context3.prev = 30;
+            case 26:
+              _context3.prev = 26;
               _context3.t1 = _context3["catch"](9);
               return _context3.abrupt("return", {
                 error: "Failed to write ".concat(pageType)
               });
-            case 33:
+            case 29:
             case "end":
               return _context3.stop();
           }
-        }, _callee3, this, [[1, 5], [9, 30]]);
+        }, _callee3, this, [[1, 5], [9, 26]]);
       }));
       function writeData(_x6) {
         return _writeData.apply(this, arguments);
@@ -298,28 +284,80 @@ var ConfluenceADFWriteProvider = /*#__PURE__*/function () {
       return writeData;
     }()
   }, {
-    key: "deleteData",
+    key: "createData",
     value: function () {
-      var _deleteData = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee4(resourceId) {
-        var deletePayload, deleteResult, match, _match2, pageId, pageType, localId, key, options, _stringifyError;
+      var _createData = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee4(syncBlockData) {
+        var match, resourceId, _match2, pageId, pageType, localId, key, sourceAri, syncBlockDataWithSourceDocumentAri;
         return _regenerator.default.wrap(function _callee4$(_context4) {
           while (1) switch (_context4.prev = _context4.next) {
             case 0:
-              _context4.prev = 0;
+              resourceId = syncBlockData.resourceId;
+              _context4.prev = 1;
               match = (0, _ari.getPageIdAndTypeFromConfluencePageAri)(resourceId);
-              _context4.next = 7;
+              _context4.next = 8;
               break;
-            case 4:
-              _context4.prev = 4;
-              _context4.t0 = _context4["catch"](0);
+            case 5:
+              _context4.prev = 5;
+              _context4.t0 = _context4["catch"](1);
+              return _context4.abrupt("return", {
+                error: (0, _errorHandling.stringifyError)(_context4.t0)
+              });
+            case 8:
+              _match2 = match, pageId = _match2.id, pageType = _match2.type;
+              _context4.prev = 9;
+              localId = (0, _ari.getLocalIdFromConfluencePageAri)(resourceId);
+              key = getContentPropertyKey(this.config.contentPropertyKey, localId);
+              sourceAri = (0, _ari.getConfluencePageAri)(pageId, this.config.cloudId, pageType);
+              syncBlockDataWithSourceDocumentAri = _objectSpread(_objectSpread({}, syncBlockData), {}, {
+                product: 'confluence-page',
+                sourceAri: sourceAri
+              });
+              _context4.next = 16;
+              return this.createNewContentProperty(pageId, key, syncBlockDataWithSourceDocumentAri, pageType);
+            case 16:
               return _context4.abrupt("return", {
+                resourceId: resourceId
+              });
+            case 19:
+              _context4.prev = 19;
+              _context4.t1 = _context4["catch"](9);
+              return _context4.abrupt("return", Promise.resolve({
+                error: (0, _errorHandling.stringifyError)(_context4.t1)
+              }));
+            case 22:
+            case "end":
+              return _context4.stop();
+          }
+        }, _callee4, this, [[1, 5], [9, 19]]);
+      }));
+      function createData(_x7) {
+        return _createData.apply(this, arguments);
+      }
+      return createData;
+    }()
+  }, {
+    key: "deleteData",
+    value: function () {
+      var _deleteData = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee5(resourceId) {
+        var deletePayload, deleteResult, match, _match3, pageId, pageType, localId, key, options, _stringifyError;
+        return _regenerator.default.wrap(function _callee5$(_context5) {
+          while (1) switch (_context5.prev = _context5.next) {
+            case 0:
+              _context5.prev = 0;
+              match = (0, _ari.getPageIdAndTypeFromConfluencePageAri)(resourceId);
+              _context5.next = 7;
+              break;
+            case 4:
+              _context5.prev = 4;
+              _context5.t0 = _context5["catch"](0);
+              return _context5.abrupt("return", {
                 resourceId: resourceId,
                 success: false,
-                error: (0, _errorHandling.stringifyError)(_context4.t0)
+                error: (0, _errorHandling.stringifyError)(_context5.t0)
               });
             case 7:
-              _match2 = match, pageId = _match2.id, pageType = _match2.type;
-              _context4.prev = 8;
+              _match3 = match, pageId = _match3.id, pageType = _match3.type;
+              _context5.prev = 8;
               localId = (0, _ari.getLocalIdFromConfluencePageAri)(resourceId);
               key = getContentPropertyKey(this.config.contentPropertyKey, localId);
               options = {
@@ -328,34 +366,34 @@ var ConfluenceADFWriteProvider = /*#__PURE__*/function () {
                 cloudId: this.config.cloudId,
                 pageType: pageType
               };
-              _context4.next = 14;
+              _context5.next = 14;
               return (0, _contentProperty3.deleteContentProperty)(options);
             case 14:
-              deletePayload = _context4.sent;
+              deletePayload = _context5.sent;
               deleteResult = (0, _utils.isBlogPageType)(pageType) ? deletePayload.data.confluence.deleteBlogPostProperty : deletePayload.data.confluence.deletePageProperty;
-              _context4.next = 21;
+              _context5.next = 21;
               break;
             case 18:
-              _context4.prev = 18;
-              _context4.t1 = _context4["catch"](8);
-              return _context4.abrupt("return", {
+              _context5.prev = 18;
+              _context5.t1 = _context5["catch"](8);
+              return _context5.abrupt("return", {
                 resourceId: resourceId,
                 success: false,
-                error: (_stringifyError = (0, _errorHandling.stringifyError)(_context4.t1)) !== null && _stringifyError !== void 0 ? _stringifyError : "Fail to delete ".concat(pageType, " content property")
+                error: (_stringifyError = (0, _errorHandling.stringifyError)(_context5.t1)) !== null && _stringifyError !== void 0 ? _stringifyError : "Fail to delete ".concat(pageType, " content property")
               });
             case 21:
-              return _context4.abrupt("return", {
+              return _context5.abrupt("return", {
                 resourceId: resourceId,
                 success: deleteResult.success,
                 error: deleteResult.errors.join()
               });
             case 22:
             case "end":
-              return _context4.stop();
+              return _context5.stop();
           }
-        }, _callee4, this, [[0, 4], [8, 18]]);
+        }, _callee5, this, [[0, 4], [8, 18]]);
       }));
-      function deleteData(_x7) {
+      function deleteData(_x8) {
         return _deleteData.apply(this, arguments);
       }
       return deleteData;

package/dist/cjs/providers/syncBlockProvider.js

@@ -150,7 +150,19 @@ var SyncBlockProvider = exports.SyncBlockProvider = /*#__PURE__*/function (_Sync
         return _writeNodesData.apply(this, arguments);
       }
       return writeNodesData;
-    }()
+    }())
+  }, {
+    key: "createNodeData",
+    value: function createNodeData(data) {
+      return this.writeProvider.createData(data).then(function (result) {
+        return result;
+      }, function (error) {
+        return {
+          error: error
+        };
+      });
+    }
+
     /**
      * Delete the data from the write provider
      *
@@ -158,7 +170,6 @@ var SyncBlockProvider = exports.SyncBlockProvider = /*#__PURE__*/function (_Sync
      *
      * @returns Array of {resourceId?: string, error?: string}.
      */
-    )
   }, {
     key: "deleteNodesData",
     value: (function () {

package/dist/cjs/store-manager/referenceSyncBlockStoreManager.js

@@ -233,17 +233,15 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
                   (_this2$fireAnalyticsE = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE === void 0 || _this2$fireAnalyticsE.call(_this2, (0, _errorHandling.fetchErrorPayload)(syncBlockInstance.error || 'Returned sync block instance does not have resource id'));
                   return;
                 }
+                var existingSyncBlock = _this2.getFromCache(syncBlockInstance.resourceId);
+                var resolvedSyncBlockInstance = existingSyncBlock ? (0, _resolveSyncBlockInstance.resolveSyncBlockInstance)(existingSyncBlock, syncBlockInstance) : syncBlockInstance;
+                _this2.updateCache(resolvedSyncBlockInstance);
+                resolvedData.push(resolvedSyncBlockInstance);
                 if (syncBlockInstance.error) {
                   var _this2$fireAnalyticsE2;
                   (_this2$fireAnalyticsE2 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE2 === void 0 || _this2$fireAnalyticsE2.call(_this2, (0, _errorHandling.fetchErrorPayload)(syncBlockInstance.error));
-                  _this2.updateCache(syncBlockInstance);
-                  resolvedData.push(syncBlockInstance);
                   return;
                 }
-                var existingSyncBlock = _this2.getFromCache(syncBlockInstance.resourceId);
-                var resolvedSyncBlockInstance = existingSyncBlock ? (0, _resolveSyncBlockInstance.resolveSyncBlockInstance)(existingSyncBlock, syncBlockInstance) : syncBlockInstance;
-                _this2.updateCache(resolvedSyncBlockInstance);
-                resolvedData.push(resolvedSyncBlockInstance);
                 _this2.fetchSyncBlockSourceInfo(resolvedSyncBlockInstance.resourceId);
               });
               return _context3.abrupt("return", [].concat(resolvedData, blocksWithNotFoundError));

package/dist/cjs/store-manager/sourceSyncBlockStoreManager.js

@@ -240,21 +240,19 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
         }
         var resourceId = attrs.resourceId,
           blockInstanceId = attrs.localId;
-        this.dataProvider.writeNodesData([(0, _utils.createBodiedSyncBlockNode)(blockInstanceId, resourceId)], [{
+        this.dataProvider.createNodeData({
           content: [],
           blockInstanceId: blockInstanceId,
           resourceId: resourceId
-        }]).then(function (results) {
-          results.forEach(function (result) {
-            var resourceId = result.resourceId;
-            if (resourceId) {
-              _this4.commitPendingCreation(true);
-            } else {
-              var _this4$fireAnalyticsE;
-              _this4.commitPendingCreation(false);
-              (_this4$fireAnalyticsE = _this4.fireAnalyticsEvent) === null || _this4$fireAnalyticsE === void 0 || _this4$fireAnalyticsE.call(_this4, (0, _errorHandling.createErrorPayload)(result.error || 'Failed to create bodied sync block'));
-            }
-          });
+        }).then(function (result) {
+          var resourceId = result.resourceId;
+          if (resourceId) {
+            _this4.commitPendingCreation(true);
+          } else {
+            var _this4$fireAnalyticsE;
+            _this4.commitPendingCreation(false);
+            (_this4$fireAnalyticsE = _this4.fireAnalyticsEvent) === null || _this4$fireAnalyticsE === void 0 || _this4$fireAnalyticsE.call(_this4, (0, _errorHandling.createErrorPayload)(result.error || 'Failed to create bodied sync block'));
+          }
         }).catch(function (error) {
           var _this4$fireAnalyticsE2;
           _this4.commitPendingCreation(false);

package/dist/cjs/utils/resolveSyncBlockInstance.js

@@ -6,6 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.resolveSyncBlockInstance = void 0;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
+var _types = require("../common/types");
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 /**
@@ -20,13 +21,16 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
  */
 var resolveSyncBlockInstance = exports.resolveSyncBlockInstance = function resolveSyncBlockInstance(oldResult, newResult) {
   var _newResult$data, _oldResult$data, _newResult$data2, _oldResult$data2;
-  // if the old result has no data, we simple return the new result
+  // if the old result has no data, we simply return the new result
   if (!oldResult.data) {
     return newResult;
   } else if (!newResult.data) {
-    // if the new result has no data, we simply return the old result
-    // TODO: EDITOR-2533 - handle this case based on the error type and whether we should keep old data or not
-    return oldResult;
+    // return the old result if there was an error, e.g. network error, but not if not found or forbidden
+    if (newResult.error === _types.SyncBlockError.Errored) {
+      return oldResult;
+    } else {
+      return newResult;
+    }
   }
 
   // otherwise, we merge the two results, preserving the sourceURL and sourceTitle from the old result if it exists

package/dist/cjs/utils/utils.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.createSyncBlockNode = exports.createBodiedSyncBlockNode = exports.convertSyncBlockPMNodeToSyncBlockData = exports.convertSyncBlockJSONNodeToSyncBlockNode = exports.convertPMNodesToSyncBlockNodes = exports.convertPMNodeToSyncBlockNode = void 0;
+exports.createSyncBlockNode = exports.convertSyncBlockPMNodeToSyncBlockData = exports.convertSyncBlockJSONNodeToSyncBlockNode = exports.convertPMNodesToSyncBlockNodes = exports.convertPMNodeToSyncBlockNode = void 0;
 var convertSyncBlockPMNodeToSyncBlockData = exports.convertSyncBlockPMNodeToSyncBlockData = function convertSyncBlockPMNodeToSyncBlockData(node) {
   return {
     blockInstanceId: node.attrs.localId,
@@ -20,15 +20,6 @@ var createSyncBlockNode = exports.createSyncBlockNode = function createSyncBlock
     }
   };
 };
-var createBodiedSyncBlockNode = exports.createBodiedSyncBlockNode = function createBodiedSyncBlockNode(localId, resourceId) {
-  return {
-    type: 'bodiedSyncBlock',
-    attrs: {
-      localId: localId,
-      resourceId: resourceId
-    }
-  };
-};
 var convertSyncBlockJSONNodeToSyncBlockNode = exports.convertSyncBlockJSONNodeToSyncBlockNode = function convertSyncBlockJSONNodeToSyncBlockNode(node) {
   if (node.type !== 'syncBlock' || !node.attrs || !('localId' in node.attrs) || !('resourceId' in node.attrs) || typeof node.attrs.localId !== 'string' || typeof node.attrs.resourceId !== 'string') {
     return undefined;

package/dist/es2019/providers/block-service/blockServiceAPI.js

@@ -138,21 +138,38 @@ class BlockServiceADFWriteProvider {
       };
     } catch (error) {
       if (error instanceof BlockError) {
-        if (error.status === 404) {
-          // Create the block
-          await createSyncedBlock({
-            blockAri: resourceId,
-            blockInstanceId: data.blockInstanceId,
-            sourceAri: this.sourceAri,
-            product: this.product,
-            content: JSON.stringify(data.content)
-          });
-        } else {
-          return {
-            error: mapBlockError(error),
-            resourceId
-          };
-        }
+        return {
+          error: mapBlockError(error),
+          resourceId
+        };
+      }
+      return {
+        error: stringifyError(error),
+        resourceId
+      };
+    }
+  }
+  async createData(data) {
+    const {
+      resourceId
+    } = data;
+    try {
+      await createSyncedBlock({
+        blockAri: resourceId,
+        blockInstanceId: data.blockInstanceId,
+        sourceAri: this.sourceAri,
+        product: this.product,
+        content: JSON.stringify(data.content)
+      });
+      return {
+        resourceId
+      };
+    } catch (error) {
+      if (error instanceof BlockError) {
+        return {
+          error: mapBlockError(error),
+          resourceId
+        };
       }
       return {
         error: stringifyError(error),

package/dist/es2019/providers/confluence/confluenceContentAPI.js

@@ -182,16 +182,6 @@ class ConfluenceADFWriteProvider {
         return {
           resourceId
         };
-      } else if (!updateResult) {
-        return this.createNewContentProperty(pageId, key, syncBlockDataWithSourceDocumentAri, pageType).then(() => {
-          return {
-            resourceId
-          };
-        }, error => {
-          return {
-            error
-          };
-        });
       } else {
         return {
           error: `Failed to update ${pageType} content property`
@@ -203,6 +193,41 @@ class ConfluenceADFWriteProvider {
       };
     }
   }
+  async createData(syncBlockData) {
+    let match;
+    const {
+      resourceId
+    } = syncBlockData;
+    try {
+      match = getPageIdAndTypeFromConfluencePageAri(resourceId);
+    } catch (error) {
+      return {
+        error: stringifyError(error)
+      };
+    }
+    const {
+      id: pageId,
+      type: pageType
+    } = match;
+    try {
+      const localId = getLocalIdFromConfluencePageAri(resourceId);
+      const key = getContentPropertyKey(this.config.contentPropertyKey, localId);
+      const sourceAri = getConfluencePageAri(pageId, this.config.cloudId, pageType);
+      const syncBlockDataWithSourceDocumentAri = {
+        ...syncBlockData,
+        product: 'confluence-page',
+        sourceAri
+      };
+      await this.createNewContentProperty(pageId, key, syncBlockDataWithSourceDocumentAri, pageType);
+      return {
+        resourceId
+      };
+    } catch (error) {
+      return Promise.resolve({
+        error: stringifyError(error)
+      });
+    }
+  }
   async deleteData(resourceId) {
     let deletePayload, deleteResult, match;
     try {

package/dist/es2019/providers/syncBlockProvider.js

@@ -98,6 +98,11 @@ export class SyncBlockProvider extends SyncBlockDataProvider {
       }
     });
   }
+  createNodeData(data) {
+    return this.writeProvider.createData(data).then(result => result, error => ({
+      error
+    }));
+  }
 
   /**
    * Delete the data from the write provider

package/dist/es2019/store-manager/referenceSyncBlockStoreManager.js

@@ -139,17 +139,15 @@ export class ReferenceSyncBlockStoreManager {
         (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 ? void 0 : _this$fireAnalyticsEv5.call(this, fetchErrorPayload(syncBlockInstance.error || 'Returned sync block instance does not have resource id'));
         return;
       }
+      const existingSyncBlock = this.getFromCache(syncBlockInstance.resourceId);
+      const resolvedSyncBlockInstance = existingSyncBlock ? resolveSyncBlockInstance(existingSyncBlock, syncBlockInstance) : syncBlockInstance;
+      this.updateCache(resolvedSyncBlockInstance);
+      resolvedData.push(resolvedSyncBlockInstance);
       if (syncBlockInstance.error) {
         var _this$fireAnalyticsEv6;
         (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 ? void 0 : _this$fireAnalyticsEv6.call(this, fetchErrorPayload(syncBlockInstance.error));
-        this.updateCache(syncBlockInstance);
-        resolvedData.push(syncBlockInstance);
         return;
       }
-      const existingSyncBlock = this.getFromCache(syncBlockInstance.resourceId);
-      const resolvedSyncBlockInstance = existingSyncBlock ? resolveSyncBlockInstance(existingSyncBlock, syncBlockInstance) : syncBlockInstance;
-      this.updateCache(resolvedSyncBlockInstance);
-      resolvedData.push(resolvedSyncBlockInstance);
       this.fetchSyncBlockSourceInfo(resolvedSyncBlockInstance.resourceId);
     });
     return [...resolvedData, ...blocksWithNotFoundError];