@atlaskit/editor-synced-block-provider 2.10.6 → 2.11.0

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @atlaskit/editor-synced-block-provider
 
+## 2.11.0
+
+### Minor Changes
+
+- [`1ed7afa478690`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/1ed7afa478690) -
+  Update the blocks endpoint calls to match API design
+
+### Patch Changes
+
+- [`0cc1855e9071d`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/0cc1855e9071d) -
+  EDITOR-1921 add analytics for sync block provider failures
+- Updated dependencies
+
 ## 2.10.6
 
 ### Patch Changes

package/dist/cjs/clients/block-service/blockService.js

@@ -39,7 +39,7 @@ var getSyncedBlockContent = exports.getSyncedBlockContent = /*#__PURE__*/functio
         case 0:
           blockAri = _ref.blockAri;
           _context.next = 3;
-          return fetch("".concat(BLOCK_SERVICE_API_URL, "/block/").concat(blockAri), {
+          return fetch("".concat(BLOCK_SERVICE_API_URL, "/block/").concat(encodeURIComponent(blockAri)), {
             method: 'GET',
             headers: COMMON_HEADERS
           });
@@ -73,7 +73,7 @@ var deleteSyncedBlock = exports.deleteSyncedBlock = /*#__PURE__*/function () {
         case 0:
           blockAri = _ref3.blockAri;
           _context2.next = 3;
-          return fetch("".concat(BLOCK_SERVICE_API_URL, "/block/").concat(blockAri), {
+          return fetch("".concat(BLOCK_SERVICE_API_URL, "/block/").concat(encodeURIComponent(blockAri)), {
             method: 'DELETE',
             headers: COMMON_HEADERS
           });
@@ -102,7 +102,7 @@ var updateSyncedBlock = exports.updateSyncedBlock = /*#__PURE__*/function () {
         case 0:
           blockAri = _ref5.blockAri, content = _ref5.content;
           _context3.next = 3;
-          return fetch("".concat(BLOCK_SERVICE_API_URL, "/block/").concat(blockAri), {
+          return fetch("".concat(BLOCK_SERVICE_API_URL, "/block/").concat(encodeURIComponent(blockAri)), {
             method: 'PUT',
             headers: COMMON_HEADERS,
             body: JSON.stringify({
@@ -134,10 +134,11 @@ var createSyncedBlock = exports.createSyncedBlock = /*#__PURE__*/function () {
         case 0:
           blockAri = _ref7.blockAri, blockInstanceId = _ref7.blockInstanceId, sourceAri = _ref7.sourceAri, product = _ref7.product, content = _ref7.content;
           _context4.next = 3;
-          return fetch("".concat(BLOCK_SERVICE_API_URL, "/block/").concat(blockAri), {
+          return fetch("".concat(BLOCK_SERVICE_API_URL, "/block"), {
             method: 'POST',
             headers: COMMON_HEADERS,
             body: JSON.stringify({
+              blockAri: blockAri,
               blockInstanceId: blockInstanceId,
               sourceAri: sourceAri,
               product: product,

package/dist/cjs/clients/confluence/sourceInfo.js

@@ -8,6 +8,8 @@ exports.fetchConfluencePageInfo = void 0;
 var _regenerator = _interopRequireDefault(require("@babel/runtime/regenerator"));
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/asyncToGenerator"));
+var _monitoring = require("@atlaskit/editor-common/monitoring");
+var _errorHandling = require("../../utils/errorHandling");
 var _ari = require("./ari");
 var _utils = require("./utils");
 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; }
@@ -69,7 +71,7 @@ var getConfluenceSourceInfo = /*#__PURE__*/function () {
   };
 }();
 var fetchConfluencePageInfo = exports.fetchConfluencePageInfo = /*#__PURE__*/function () {
-  var _ref2 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee2(pageAri, localId) {
+  var _ref2 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee2(pageAri, localId, fireAnalyticsEvent) {
     var _response$data, _contentData$space, _getPageIdAndTypeFrom, pageType, response, contentData, title, url, _ref3, base;
     return _regenerator.default.wrap(function _callee2$(_context2) {
       while (1) switch (_context2.prev = _context2.next) {
@@ -81,15 +83,9 @@ var fetchConfluencePageInfo = exports.fetchConfluencePageInfo = /*#__PURE__*/fun
         case 4:
           response = _context2.sent;
           contentData = (_response$data = response.data) === null || _response$data === void 0 || (_response$data = _response$data.content) === null || _response$data === void 0 || (_response$data = _response$data.nodes) === null || _response$data === void 0 ? void 0 : _response$data[0];
-          if (contentData) {
-            _context2.next = 8;
-            break;
-          }
-          throw new Error("Failed to get content data");
-        case 8:
-          title = contentData.title;
-          _ref3 = contentData.links || {}, base = _ref3.base;
-          if (base && (_contentData$space = contentData.space) !== null && _contentData$space !== void 0 && _contentData$space.key && contentData.id) {
+          title = contentData === null || contentData === void 0 ? void 0 : contentData.title;
+          _ref3 = (contentData === null || contentData === void 0 ? void 0 : contentData.links) || {}, base = _ref3.base;
+          if (base && contentData !== null && contentData !== void 0 && (_contentData$space = contentData.space) !== null && _contentData$space !== void 0 && _contentData$space.key && contentData !== null && contentData !== void 0 && contentData.id) {
             if ((0, _utils.isBlogPageType)(pageType)) {
               url = "".concat(base, "/spaces/").concat(contentData.space.key, "/blog/edit-v2/").concat(contentData.id);
             } else if (contentData.subType === 'live') {
@@ -99,21 +95,28 @@ var fetchConfluencePageInfo = exports.fetchConfluencePageInfo = /*#__PURE__*/fun
             }
           }
           url = url && localId ? "".concat(url, "#block-").concat(localId) : url;
+          if (!title || !url) {
+            fireAnalyticsEvent === null || fireAnalyticsEvent === void 0 || fireAnalyticsEvent((0, _errorHandling.getSourceInfoErrorPayload)('Failed to get confluence page source info'));
+          }
           return _context2.abrupt("return", Promise.resolve({
             title: title,
             url: url
           }));
-        case 15:
-          _context2.prev = 15;
+        case 14:
+          _context2.prev = 14;
           _context2.t0 = _context2["catch"](0);
+          (0, _monitoring.logException)(_context2.t0, {
+            location: 'editor-synced-block-provider/sourceInfo'
+          });
+          fireAnalyticsEvent === null || fireAnalyticsEvent === void 0 || fireAnalyticsEvent((0, _errorHandling.getSourceInfoErrorPayload)(_context2.t0.message));
           return _context2.abrupt("return", Promise.resolve(undefined));
-        case 18:
+        case 19:
         case "end":
           return _context2.stop();
       }
-    }, _callee2, null, [[0, 15]]);
+    }, _callee2, null, [[0, 14]]);
   }));
-  return function fetchConfluencePageInfo(_x2, _x3) {
+  return function fetchConfluencePageInfo(_x2, _x3, _x4) {
     return _ref2.apply(this, arguments);
   };
 }();

package/dist/cjs/hooks/useFetchSyncBlockData.js

@@ -9,9 +9,11 @@ var _regenerator = _interopRequireDefault(require("@babel/runtime/regenerator"))
 var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/asyncToGenerator"));
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _react = require("react");
+var _monitoring = require("@atlaskit/editor-common/monitoring");
 var _types = require("../common/types");
+var _errorHandling = require("../utils/errorHandling");
 var _utils = require("../utils/utils");
-var useFetchSyncBlockData = exports.useFetchSyncBlockData = function useFetchSyncBlockData(manager, resourceId, localId) {
+var useFetchSyncBlockData = exports.useFetchSyncBlockData = function useFetchSyncBlockData(manager, resourceId, localId, fireAnalyticsEvent) {
   var _useState = (0, _react.useState)(null),
     _useState2 = (0, _slicedToArray2.default)(_useState, 2),
     syncBlockInstance = _useState2[0],
@@ -31,36 +33,43 @@ var useFetchSyncBlockData = exports.useFetchSyncBlockData = function useFetchSyn
           }
           return _context.abrupt("return");
         case 2:
+          _context.prev = 2;
           syncBlockNode = resourceId && localId ? (0, _utils.createSyncBlockNode)(localId, resourceId) : null;
           if (syncBlockNode) {
-            _context.next = 5;
+            _context.next = 6;
             break;
           }
-          return _context.abrupt("return");
-        case 5:
+          throw new Error('Failed to create sync block node from resourceid and localid');
+        case 6:
           setIsLoading(true);
-          _context.prev = 6;
+
+          // Fetch sync block data, the `subscribeToSyncBlock` will update the state once data is fetched
           _context.next = 9;
           return manager.referenceManager.fetchSyncBlocksData([syncBlockNode]);
         case 9:
-          _context.next = 14;
+          _context.next = 16;
           break;
         case 11:
           _context.prev = 11;
-          _context.t0 = _context["catch"](6);
+          _context.t0 = _context["catch"](2);
+          (0, _monitoring.logException)(_context.t0, {
+            location: 'editor-synced-block-provider/useFetchSyncBlockData'
+          });
+          fireAnalyticsEvent === null || fireAnalyticsEvent === void 0 || fireAnalyticsEvent((0, _errorHandling.fetchErrorPayload)(_context.t0.message));
+
           // Set error state if fetching fails
           setSyncBlockInstance({
             resourceId: resourceId || '',
             error: _types.SyncBlockError.Errored
           });
-        case 14:
+        case 16:
           setIsLoading(false);
-        case 15:
+        case 17:
         case "end":
           return _context.stop();
       }
-    }, _callee, null, [[6, 11]]);
-  })), [isLoading, localId, manager.referenceManager, resourceId]);
+    }, _callee, null, [[2, 11]]);
+  })), [isLoading, localId, manager.referenceManager, resourceId, fireAnalyticsEvent]);
   (0, _react.useEffect)(function () {
     var unsubscribe = manager.referenceManager.subscribeToSyncBlock(resourceId || '', localId || '', function (data) {
       setSyncBlockInstance(data);

package/dist/cjs/hooks/useHandleContentChanges.js

@@ -7,10 +7,6 @@ exports.useHandleContentChanges = void 0;
 var _react = require("react");
 var useHandleContentChanges = exports.useHandleContentChanges = function useHandleContentChanges(manager, syncBlockNode) {
   (0, _react.useEffect)(function () {
-    try {
-      manager.sourceManager.updateSyncBlockData(syncBlockNode);
-    } catch (_unused) {
-      //TODO: EDITOR-1921 - add error analytics
-    }
+    manager.sourceManager.updateSyncBlockData(syncBlockNode);
   }, [manager, syncBlockNode]);
 };

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

@@ -102,13 +102,16 @@ var BlockServiceADFFetchProvider = /*#__PURE__*/function () {
  * ADFWriteProvider implementation that writes synced block data to Block Service API
  */
 var BlockServiceADFWriteProvider = /*#__PURE__*/function () {
-  function BlockServiceADFWriteProvider() {
+  function BlockServiceADFWriteProvider(sourceAri, product) {
     (0, _classCallCheck2.default)(this, BlockServiceADFWriteProvider);
+    this.sourceAri = sourceAri;
+    this.product = product;
   }
+
+  // it will first try to update and if it can't (404) then it will try to create
   return (0, _createClass2.default)(BlockServiceADFWriteProvider, [{
     key: "writeData",
-    value: // it will first try to update and if it can't (404) then it will try to create
-    function () {
+    value: function () {
       var _writeData = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee2(data) {
         var resourceId;
         return _regenerator.default.wrap(function _callee2$(_context2) {
@@ -140,8 +143,8 @@ var BlockServiceADFWriteProvider = /*#__PURE__*/function () {
               return (0, _blockService.createSyncedBlock)({
                 blockAri: resourceId,
                 blockInstanceId: data.blockInstanceId,
-                sourceAri: resourceId,
-                product: 'confluence-page',
+                sourceAri: this.sourceAri,
+                product: this.product,
                 content: JSON.stringify(data.content)
               });
             case 13:
@@ -161,7 +164,7 @@ var BlockServiceADFWriteProvider = /*#__PURE__*/function () {
             case "end":
               return _context2.stop();
           }
-        }, _callee2, null, [[1, 7]]);
+        }, _callee2, this, [[1, 7]]);
       }));
       function writeData(_x2) {
         return _writeData.apply(this, arguments);
@@ -217,22 +220,24 @@ var BlockServiceADFWriteProvider = /*#__PURE__*/function () {
     }()
   }, {
     key: "generateResourceId",
-    value: function generateResourceId(sourceId, localId) {
-      return (0, _ari.blockResourceIdFromSourceAndLocalId)(sourceId, localId);
+    value: function generateResourceId(sourceAri, localId) {
+      return (0, _ari.blockResourceIdFromSourceAndLocalId)(sourceAri, localId);
     }
   }]);
 }();
 /**
  * Factory function to create both providers with shared configuration
  */
-var createBlockServiceAPIProviders = function createBlockServiceAPIProviders() {
+var createBlockServiceAPIProviders = function createBlockServiceAPIProviders(sourceAri, product) {
   var fetchProvider = new BlockServiceADFFetchProvider();
-  var writeProvider = new BlockServiceADFWriteProvider();
+  var writeProvider = new BlockServiceADFWriteProvider(sourceAri, product);
   return {
     fetchProvider: fetchProvider,
     writeProvider: writeProvider
   };
 };
-var useMemoizedBlockServiceAPIProviders = exports.useMemoizedBlockServiceAPIProviders = function useMemoizedBlockServiceAPIProviders() {
-  return (0, _react.useMemo)(createBlockServiceAPIProviders, []);
+var useMemoizedBlockServiceAPIProviders = exports.useMemoizedBlockServiceAPIProviders = function useMemoizedBlockServiceAPIProviders(sourceAri, product) {
+  return (0, _react.useMemo)(function () {
+    return createBlockServiceAPIProviders(sourceAri, product);
+  }, [sourceAri, product]);
 };

package/dist/cjs/providers/syncBlockProvider.js

@@ -91,7 +91,7 @@ var SyncBlockProvider = exports.SyncBlockProvider = /*#__PURE__*/function (_Sync
           return data;
         }, function () {
           return {
-            status: _types.SyncBlockError.Errored,
+            error: _types.SyncBlockError.Errored,
             resourceId: resourceId
           };
         });
@@ -219,13 +219,13 @@ var SyncBlockProvider = exports.SyncBlockProvider = /*#__PURE__*/function (_Sync
      */
   }, {
     key: "fetchSyncBlockSourceInfo",
-    value: function fetchSyncBlockSourceInfo(localId, sourceAri, sourceProduct) {
+    value: function fetchSyncBlockSourceInfo(localId, sourceAri, sourceProduct, fireAnalyticsEvent) {
       if (!sourceAri || !sourceProduct) {
-        return Promise.resolve(undefined);
+        return Promise.reject(new Error('Source ari or source product is undefined'));
       }
       switch (sourceProduct) {
         case 'confluence-page':
-          return (0, _sourceInfo.fetchConfluencePageInfo)(sourceAri, localId);
+          return (0, _sourceInfo.fetchConfluencePageInfo)(sourceAri, localId, fireAnalyticsEvent);
         case 'jira-work-item':
           return Promise.reject(new Error('Jira work item source product not supported'));
         default:

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

@@ -11,8 +11,10 @@ var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/
 var _classCallCheck2 = _interopRequireDefault(require("@babel/runtime/helpers/classCallCheck"));
 var _createClass2 = _interopRequireDefault(require("@babel/runtime/helpers/createClass"));
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
+var _monitoring = require("@atlaskit/editor-common/monitoring");
 var _providerFactory = require("@atlaskit/editor-common/provider-factory");
 var _types = require("../common/types");
+var _errorHandling = require("../utils/errorHandling");
 var _resolveSyncBlockInstance = require("../utils/resolveSyncBlockInstance");
 var _utils = require("../utils/utils");
 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; }
@@ -26,7 +28,7 @@ function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length)
 // Handles fetching source URL and title for sync blocks.
 // Can be used in both editor and renderer contexts.
 var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
-  function ReferenceSyncBlockStoreManager(dataProvider) {
+  function ReferenceSyncBlockStoreManager(dataProvider, fireAnalyticsEvent) {
     (0, _classCallCheck2.default)(this, ReferenceSyncBlockStoreManager);
     (0, _defineProperty2.default)(this, "isRefreshingSubscriptions", false);
     this.syncBlockCache = new Map();
@@ -35,17 +37,23 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
     this.dataProvider = dataProvider;
     this.syncBlockURLRequests = new Map();
     this.providerFactories = new Map();
+    this.fireAnalyticsEvent = fireAnalyticsEvent;
   }
-
-  /**
-   * Refreshes the subscriptions for all sync blocks.
-   * @returns {Promise<void>}
-   */
   return (0, _createClass2.default)(ReferenceSyncBlockStoreManager, [{
+    key: "updateFireAnalyticsEvent",
+    value: function updateFireAnalyticsEvent(fireAnalyticsEvent) {
+      this.fireAnalyticsEvent = fireAnalyticsEvent;
+    }
+
+    /**
+     * Refreshes the subscriptions for all sync blocks.
+     * @returns {Promise<void>}
+     */
+  }, {
     key: "refreshSubscriptions",
     value: (function () {
       var _refreshSubscriptions = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee() {
-        var syncBlocks, _iterator, _step, _loop;
+        var syncBlocks, _iterator, _step, _loop, _this$fireAnalyticsEv;
         return _regenerator.default.wrap(function _callee$(_context2) {
           while (1) switch (_context2.prev = _context2.next) {
             case 0:
@@ -100,20 +108,24 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
               _context2.next = 23;
               return this.fetchSyncBlocksData(syncBlocks);
             case 23:
-              _context2.next = 27;
+              _context2.next = 29;
               break;
             case 25:
               _context2.prev = 25;
               _context2.t2 = _context2["catch"](20);
-            case 27:
-              _context2.prev = 27;
+              (0, _monitoring.logException)(_context2.t2, {
+                location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
+              });
+              (_this$fireAnalyticsEv = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv === void 0 || _this$fireAnalyticsEv.call(this, (0, _errorHandling.fetchErrorPayload)(_context2.t2.message));
+            case 29:
+              _context2.prev = 29;
               this.isRefreshingSubscriptions = false;
-              return _context2.finish(27);
-            case 30:
+              return _context2.finish(29);
+            case 32:
             case "end":
               return _context2.stop();
           }
-        }, _callee, this, [[5, 14, 17, 20], [20, 25, 27, 30]]);
+        }, _callee, this, [[5, 14, 17, 20], [20, 25, 29, 32]]);
       }));
       function refreshSubscriptions() {
         return _refreshSubscriptions.apply(this, arguments);
@@ -123,43 +135,53 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
   }, {
     key: "fetchSyncBlockSourceInfo",
     value: function fetchSyncBlockSourceInfo(resourceId) {
-      var _existingSyncBlock$da,
-        _existingSyncBlock$da2,
-        _this = this;
-      if (!resourceId || !this.dataProvider) {
-        return;
-      }
-      if (this.syncBlockURLRequests.get(resourceId)) {
-        return;
-      }
-      this.syncBlockURLRequests.set(resourceId, true);
-      var existingSyncBlock = this.getFromCache(resourceId);
-      if (!existingSyncBlock) {
-        return;
-      }
+      var _this = this;
+      try {
+        var _existingSyncBlock$da, _existingSyncBlock$da2;
+        if (!resourceId || !this.dataProvider) {
+          throw new Error('Data provider or resourceId not set');
+        }
+        if (this.syncBlockURLRequests.get(resourceId)) {
+          return;
+        }
+        this.syncBlockURLRequests.set(resourceId, true);
+        var existingSyncBlock = this.getFromCache(resourceId);
+        if (!existingSyncBlock) {
+          throw new Error('No existing sync block to fetch source info for');
+        }
 
-      // skip if source URL and title are already present
-      if ((_existingSyncBlock$da = existingSyncBlock.data) !== null && _existingSyncBlock$da !== void 0 && _existingSyncBlock$da.sourceURL && (_existingSyncBlock$da2 = existingSyncBlock.data) !== null && _existingSyncBlock$da2 !== void 0 && _existingSyncBlock$da2.sourceTitle) {
-        return;
-      }
-      var _ref = existingSyncBlock.data || {},
-        sourceAri = _ref.sourceAri,
-        product = _ref.product,
-        blockInstanceId = _ref.blockInstanceId;
-      if (!sourceAri || !product || !blockInstanceId) {
-        return;
-      }
-      this.dataProvider.fetchSyncBlockSourceInfo(blockInstanceId, sourceAri, product).then(function (sourceInfo) {
-        if (!sourceInfo) {
+        // skip if source URL and title are already present
+        if ((_existingSyncBlock$da = existingSyncBlock.data) !== null && _existingSyncBlock$da !== void 0 && _existingSyncBlock$da.sourceURL && (_existingSyncBlock$da2 = existingSyncBlock.data) !== null && _existingSyncBlock$da2 !== void 0 && _existingSyncBlock$da2.sourceTitle) {
           return;
         }
-        _this.updateCacheWithSourceInfo(resourceId, sourceInfo);
-        if (sourceInfo.title) {
-          _this.updateSourceTitleSubscriptions(resourceId, sourceInfo.title);
+        var _ref = existingSyncBlock.data || {},
+          sourceAri = _ref.sourceAri,
+          product = _ref.product,
+          blockInstanceId = _ref.blockInstanceId;
+        if (!sourceAri || !product || !blockInstanceId) {
+          throw new Error('SourceAri, product or blockInstanceId missing');
         }
-      }).finally(function () {
-        _this.syncBlockURLRequests.delete(resourceId);
-      });
+        this.dataProvider.fetchSyncBlockSourceInfo(blockInstanceId, sourceAri, product, this.fireAnalyticsEvent).then(function (sourceInfo) {
+          if (!sourceInfo) {
+            return;
+          }
+          _this.updateCacheWithSourceInfo(resourceId, sourceInfo);
+          if (sourceInfo.title) {
+            _this.updateSourceTitleSubscriptions(resourceId, sourceInfo.title);
+          }
+        }).catch(function (error) {
+          var _this$fireAnalyticsEv2;
+          (_this$fireAnalyticsEv2 = _this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv2 === void 0 || _this$fireAnalyticsEv2.call(_this, (0, _errorHandling.getSourceInfoErrorPayload)(error.message));
+        }).finally(function () {
+          _this.syncBlockURLRequests.delete(resourceId);
+        });
+      } catch (error) {
+        var _this$fireAnalyticsEv3;
+        (0, _monitoring.logException)(error, {
+          location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
+        });
+        (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 || _this$fireAnalyticsEv3.call(this, (0, _errorHandling.getSourceInfoErrorPayload)(error.message));
+      }
     }
 
     /**
@@ -202,18 +224,16 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
               return this.dataProvider.fetchNodesData(nodesToFetch);
             case 8:
               data = _context3.sent;
-              if (data) {
-                _context3.next = 11;
-                break;
-              }
-              throw new Error('Failed to fetch sync block node data');
-            case 11:
               resolvedData = [];
               data.forEach(function (syncBlockInstance) {
                 if (!syncBlockInstance.resourceId) {
+                  var _this2$fireAnalyticsE;
+                  (_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;
                 }
                 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;
@@ -225,7 +245,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
                 _this2.fetchSyncBlockSourceInfo(resolvedSyncBlockInstance.resourceId);
               });
               return _context3.abrupt("return", [].concat(resolvedData, blocksWithNotFoundError));
-            case 14:
+            case 12:
             case "end":
               return _context3.stop();
           }
@@ -297,7 +317,13 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
       if (cachedData) {
         callback(cachedData);
       } else {
-        this.fetchSyncBlocksData([(0, _utils.createSyncBlockNode)(localId, resourceId)]).catch(function () {});
+        this.fetchSyncBlocksData([(0, _utils.createSyncBlockNode)(localId, resourceId)]).catch(function (error) {
+          var _this3$fireAnalyticsE;
+          (0, _monitoring.logException)(error, {
+            location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
+          });
+          (_this3$fireAnalyticsE = _this3.fireAnalyticsEvent) === null || _this3$fireAnalyticsE === void 0 || _this3$fireAnalyticsE.call(_this3, (0, _errorHandling.fetchErrorPayload)(error.message));
+        });
       }
       return function () {
         var resourceSubscriptions = _this3.subscriptions.get(resourceId);
@@ -350,17 +376,26 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
   }, {
     key: "subscribe",
     value: function subscribe(node, callback) {
-      // check node is a sync block, as we only support sync block subscriptions
-      if (node.type.name !== 'syncBlock') {
-        return function () {};
-      }
-      var _node$attrs2 = node.attrs,
-        resourceId = _node$attrs2.resourceId,
-        localId = _node$attrs2.localId;
-      if (!localId || !resourceId) {
+      try {
+        // check node is a sync block, as we only support sync block subscriptions
+        if (node.type.name !== 'syncBlock') {
+          throw new Error('Only sync block node subscriptions are supported');
+        }
+        var _node$attrs2 = node.attrs,
+          resourceId = _node$attrs2.resourceId,
+          localId = _node$attrs2.localId;
+        if (!localId || !resourceId) {
+          throw new Error('Missing local id or resource id');
+        }
+        return this.subscribeToSyncBlock(resourceId, localId, callback);
+      } catch (error) {
+        var _this$fireAnalyticsEv4;
+        (0, _monitoring.logException)(error, {
+          location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
+        });
+        (_this$fireAnalyticsEv4 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv4 === void 0 || _this$fireAnalyticsEv4.call(this, (0, _errorHandling.fetchErrorPayload)(error.message));
         return function () {};
       }
-      return this.subscribeToSyncBlock(resourceId, localId, callback);
     }
 
     /**
@@ -382,6 +417,12 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
     key: "getProviderFactory",
     value: function getProviderFactory(resourceId) {
       if (!this.dataProvider) {
+        var _this$fireAnalyticsEv5;
+        var error = new Error('Data provider not set');
+        (0, _monitoring.logException)(error, {
+          location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
+        });
+        (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 || _this$fireAnalyticsEv5.call(this, (0, _errorHandling.fetchErrorPayload)(error.message));
         return undefined;
       }
       var _this$dataProvider$ge = this.dataProvider.getSyncedBlockRendererProviderOptions(),
@@ -397,7 +438,15 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
         this.providerFactories.set(resourceId, providerFactory);
       }
       if (providerCreator) {
-        this.retrieveDynamicProviders(resourceId, providerFactory, providerCreator);
+        try {
+          this.retrieveDynamicProviders(resourceId, providerFactory, providerCreator);
+        } catch (error) {
+          var _this$fireAnalyticsEv6;
+          (0, _monitoring.logException)(error, {
+            location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
+          });
+          (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 || _this$fireAnalyticsEv6.call(this, (0, _errorHandling.fetchErrorPayload)(error.message));
+        }
       }
       return providerFactory;
     }
@@ -406,7 +455,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
     value: function retrieveDynamicProviders(resourceId, providerFactory, providerCreator) {
       var _syncBlock$data2, _syncBlock$data3, _syncBlock$data4, _syncBlock$data5;
       if (!this.dataProvider) {
-        return;
+        throw new Error('Data provider not set');
       }
       var hasMediaProvider = providerFactory.hasProvider('mediaProvider');
       var hasEmojiProvider = providerFactory.hasProvider('emojiProvider');
@@ -415,11 +464,13 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
       }
       var syncBlock = this.getFromCache(resourceId);
       if (!syncBlock || !((_syncBlock$data2 = syncBlock.data) !== null && _syncBlock$data2 !== void 0 && _syncBlock$data2.sourceAri) || !((_syncBlock$data3 = syncBlock.data) !== null && _syncBlock$data3 !== void 0 && _syncBlock$data3.product)) {
+        var _this$fireAnalyticsEv7;
+        (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 || _this$fireAnalyticsEv7.call(this, (0, _errorHandling.fetchErrorPayload)('Sync block or source ari or product not found'));
         return;
       }
       var parentInfo = this.dataProvider.retrieveSyncBlockParentInfo((_syncBlock$data4 = syncBlock.data) === null || _syncBlock$data4 === void 0 ? void 0 : _syncBlock$data4.sourceAri, (_syncBlock$data5 = syncBlock.data) === null || _syncBlock$data5 === void 0 ? void 0 : _syncBlock$data5.product);
       if (!parentInfo) {
-        return;
+        throw new Error('Unable to retrive sync block parent info');
       }
       var contentId = parentInfo.contentId,
         contentProduct = parentInfo.contentProduct;
@@ -460,6 +511,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
         providerFactory.destroy();
       });
       this.providerFactories.clear();
+      this.fireAnalyticsEvent = undefined;
     }
   }]);
 }();