@atlaskit/reactions 21.3.3 → 21.3.4

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # @atlaskit/reactions
 
+## 21.3.4
+
+### Patch Changes
+
+- [`c3f9e9bce1c`](https://bitbucket.org/atlassian/atlassian-frontend/commits/c3f9e9bce1c) - Add custom information to failure and abort events for UFO experiences
+- Updated dependencies
+
 ## 21.3.3
 
 ### Patch Changes

package/dist/cjs/components/ReactionPicker.js

@@ -189,7 +189,12 @@ var ReactionPicker = /*#__PURE__*/function (_PureComponent) {
     value: function componentWillUnmount() {
       document.removeEventListener('click', this.handleClickOutside);
 
-      _analytics.UFO.PickerRender.abort();
+      _analytics.UFO.PickerRender.abort({
+        metadata: {
+          source: 'Reaction-Picker',
+          reason: 'unmount'
+        }
+      });
     }
   }, {
     key: "close",
@@ -199,7 +204,13 @@ var ReactionPicker = /*#__PURE__*/function (_PureComponent) {
         showFullPicker: false
       }, function () {
         // ufo abort reaction experience
-        _analytics.UFO.PickerRender.abort();
+        _analytics.UFO.PickerRender.abort({
+          metadata: {
+            emojiId: _emojiId,
+            source: 'Reaction-Picker',
+            reason: 'close dialog'
+          }
+        });
       });
     }
   }, {

package/dist/cjs/components/UfoErrorBoundary.js

@@ -41,14 +41,24 @@ var UfoErrorBoundary = /*#__PURE__*/function (_React$Component) {
 
   (0, _createClass2.default)(UfoErrorBoundary, [{
     key: "componentDidCatch",
-    value: function componentDidCatch() {
+    value: function componentDidCatch(error, errorInfo) {
       var _iterator = _createForOfIteratorHelper(this.props.experiences),
           _step;
 
       try {
         for (_iterator.s(); !(_step = _iterator.n()).done;) {
-          var e = _step.value;
-          e.failure();
+          var exp = _step.value;
+          exp.failure({
+            metadata: {
+              source: 'UfoErrorBoundary',
+              reason: 'error',
+              error: {
+                name: error.name,
+                message: error.message,
+                infoStack: errorInfo.componentStack
+              }
+            }
+          });
         }
       } catch (err) {
         _iterator.e(err);

package/dist/cjs/containers/ConnectedReactionsView.js

@@ -34,7 +34,8 @@ var ReactionsView = function ReactionsView(props) {
   var experienceInstance = (0, _react.useRef)();
   var ari = props.ari,
       createAnalyticsEvent = props.createAnalyticsEvent,
-      store = props.store;
+      store = props.store,
+      containerAri = props.containerAri;
   (0, _react.useEffect)(function () {
     experienceInstance.current = _ReactionsStore.ufoExperiences.render.getInstance(ari);
   }, [ari]);
@@ -47,16 +48,23 @@ var ReactionsView = function ReactionsView(props) {
   }, [createAnalyticsEvent, store]); // abort when component gets unmounted
 
   (0, _react.useEffect)(function () {
-    return function () {
+    return function cleanup() {
       var _experienceInstance$c;
 
-      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort();
+      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort({
+        metadata: {
+          source: 'Connected-Reactions-View',
+          data: {
+            ari: ari,
+            containerAri: containerAri
+          },
+          reason: 'unmount'
+        }
+      });
     };
-  }, [experienceInstance]);
+  }, [experienceInstance, containerAri, ari]);
 
   var renderChildren = function renderChildren(innerProps) {
-    var containerAri = props.containerAri,
-        ari = props.ari;
     return /*#__PURE__*/_react.default.createElement(_analyticsNamespacedContext.FabricElementsAnalyticsContext, {
       data: {
         containerAri: containerAri,

package/dist/cjs/store/ReactionsStore.js

@@ -166,7 +166,15 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
 
           exp.failure({
             metadata: {
-              error: error
+              error: error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri: containerAri,
+                ari: ari,
+                emojiId: emojiId,
+                metadata: _this.metadata
+              },
+              reason: 'addReaction fetch failed'
             }
           });
         }
@@ -191,7 +199,15 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
         // ufo add reaction failure
         exp.failure({
           metadata: {
-            error: error
+            error: error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri: containerAri,
+              ari: ari,
+              emojiId: emojiId,
+              metadata: _this.metadata
+            },
+            reason: 'deleteReaction fetch failed'
           }
         });
       });
@@ -204,11 +220,12 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
        * TODO:
        * All reactions are usually fetched in a single call to reactions-service. Need to check why "getReactions" gets called randomly 1-2 times everytime on each fetch request despite using same containerAri.
        */
-      var exp = ufoExperiences.render.getInstance(containerAri); // ufo start reaction experience
+      var exp = ufoExperiences.render.getInstance(containerAri);
+      var arisArr = aris.reduce(utils.flattenAris); // ufo start reaction experience
 
       exp.start();
 
-      _this.client.getReactions(containerAri, aris.reduce(utils.flattenAris)).then(function (value) {
+      _this.client.getReactions(containerAri, arisArr).then(function (value) {
         Object.keys(value).map(function (ari) {
           var reactionsState = _this.getReactionsState(containerAri, ari);
 
@@ -228,7 +245,17 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
             Analytics.createAndFireSafe(_this.createAnalyticsEvent, Analytics.createRestFailedEvent, 'getReactions', error.code);
           }
 
-          exp.failure();
+          exp.failure({
+            metadata: {
+              error: error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri: containerAri,
+                aris: arisArr.join(',')
+              },
+              reason: 'getReactions fetch failed'
+            }
+          });
           return Promise.reject(error);
         }
       });
@@ -248,7 +275,14 @@ var MemoryReactionsStore = /*#__PURE__*/function () {
         // ufo get reaction details failure
         exp.failure({
           metadata: {
-            error: error
+            error: error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri: containerAri,
+              ari: ari,
+              emojiId: emojiId
+            },
+            reason: 'getDetailedReaction fetch failed'
           }
         });
       });

package/dist/cjs/version.json

@@ -1,4 +1,4 @@
 {
 	"name": "@atlaskit/reactions",
-	"version": "21.3.3"
+	"version": "21.3.4"
 }

package/dist/es2019/components/ReactionPicker.js

@@ -143,7 +143,12 @@ export class ReactionPicker extends PureComponent {
 
   componentWillUnmount() {
     document.removeEventListener('click', this.handleClickOutside);
-    UFO.PickerRender.abort();
+    UFO.PickerRender.abort({
+      metadata: {
+        source: 'Reaction-Picker',
+        reason: 'unmount'
+      }
+    });
   }
 
   close(_emojiId) {
@@ -152,7 +157,13 @@ export class ReactionPicker extends PureComponent {
       showFullPicker: false
     }, () => {
       // ufo abort reaction experience
-      UFO.PickerRender.abort();
+      UFO.PickerRender.abort({
+        metadata: {
+          emojiId: _emojiId,
+          source: 'Reaction-Picker',
+          reason: 'close dialog'
+        }
+      });
     });
   }
 

package/dist/es2019/components/UfoErrorBoundary.js

@@ -1,8 +1,18 @@
 import React from 'react';
 export class UfoErrorBoundary extends React.Component {
-  componentDidCatch() {
-    for (const e of this.props.experiences) {
-      e.failure();
+  componentDidCatch(error, errorInfo) {
+    for (const exp of this.props.experiences) {
+      exp.failure({
+        metadata: {
+          source: 'UfoErrorBoundary',
+          reason: 'error',
+          error: {
+            name: error.name,
+            message: error.message,
+            infoStack: errorInfo.componentStack
+          }
+        }
+      });
     }
   }
 

package/dist/es2019/containers/ConnectedReactionsView.js

@@ -13,7 +13,8 @@ const ReactionsView = props => {
   const {
     ari,
     createAnalyticsEvent,
-    store
+    store,
+    containerAri
   } = props;
   useEffect(() => {
     experienceInstance.current = ufoExperiences.render.getInstance(ari);
@@ -27,18 +28,23 @@ const ReactionsView = props => {
   }, [createAnalyticsEvent, store]); // abort when component gets unmounted
 
   useEffect(() => {
-    return () => {
+    return function cleanup() {
       var _experienceInstance$c;
 
-      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort();
+      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort({
+        metadata: {
+          source: 'Connected-Reactions-View',
+          data: {
+            ari,
+            containerAri
+          },
+          reason: 'unmount'
+        }
+      });
     };
-  }, [experienceInstance]);
+  }, [experienceInstance, containerAri, ari]);
 
   const renderChildren = innerProps => {
-    const {
-      containerAri,
-      ari
-    } = props;
     return /*#__PURE__*/React.createElement(FabricElementsAnalyticsContext, {
       data: {
         containerAri,

package/dist/es2019/store/ReactionsStore.js

@@ -131,7 +131,15 @@ export class MemoryReactionsStore {
 
           exp.failure({
             metadata: {
-              error
+              error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri,
+                ari,
+                emojiId,
+                metadata: this.metadata
+              },
+              reason: 'addReaction fetch failed'
             }
           });
         }
@@ -157,7 +165,15 @@ export class MemoryReactionsStore {
         // ufo add reaction failure
         exp.failure({
           metadata: {
-            error
+            error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri,
+              ari,
+              emojiId,
+              metadata: this.metadata
+            },
+            reason: 'deleteReaction fetch failed'
           }
         });
       });
@@ -172,10 +188,11 @@ export class MemoryReactionsStore {
        * TODO:
        * All reactions are usually fetched in a single call to reactions-service. Need to check why "getReactions" gets called randomly 1-2 times everytime on each fetch request despite using same containerAri.
        */
-      const exp = ufoExperiences.render.getInstance(containerAri); // ufo start reaction experience
+      const exp = ufoExperiences.render.getInstance(containerAri);
+      const arisArr = aris.reduce(utils.flattenAris); // ufo start reaction experience
 
       exp.start();
-      this.client.getReactions(containerAri, aris.reduce(utils.flattenAris)).then(value => {
+      this.client.getReactions(containerAri, arisArr).then(value => {
         Object.keys(value).map(ari => {
           const reactionsState = this.getReactionsState(containerAri, ari);
           const reactions = reactionsState && reactionsState.status === Types.ReactionStatus.ready ? reactionsState.reactions : undefined;
@@ -193,7 +210,17 @@ export class MemoryReactionsStore {
             Analytics.createAndFireSafe(this.createAnalyticsEvent, Analytics.createRestFailedEvent, 'getReactions', error.code);
           }
 
-          exp.failure();
+          exp.failure({
+            metadata: {
+              error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri,
+                aris: arisArr.join(',')
+              },
+              reason: 'getReactions fetch failed'
+            }
+          });
           return Promise.reject(error);
         }
       });
@@ -214,7 +241,14 @@ export class MemoryReactionsStore {
         // ufo get reaction details failure
         exp.failure({
           metadata: {
-            error
+            error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri,
+              ari,
+              emojiId
+            },
+            reason: 'getDetailedReaction fetch failed'
           }
         });
       });

package/dist/es2019/version.json

@@ -1,4 +1,4 @@
 {
 	"name": "@atlaskit/reactions",
-	"version": "21.3.3"
+	"version": "21.3.4"
 }

package/dist/esm/components/ReactionPicker.js

@@ -166,7 +166,12 @@ export var ReactionPicker = /*#__PURE__*/function (_PureComponent) {
     key: "componentWillUnmount",
     value: function componentWillUnmount() {
       document.removeEventListener('click', this.handleClickOutside);
-      UFO.PickerRender.abort();
+      UFO.PickerRender.abort({
+        metadata: {
+          source: 'Reaction-Picker',
+          reason: 'unmount'
+        }
+      });
     }
   }, {
     key: "close",
@@ -176,7 +181,13 @@ export var ReactionPicker = /*#__PURE__*/function (_PureComponent) {
         showFullPicker: false
       }, function () {
         // ufo abort reaction experience
-        UFO.PickerRender.abort();
+        UFO.PickerRender.abort({
+          metadata: {
+            emojiId: _emojiId,
+            source: 'Reaction-Picker',
+            reason: 'close dialog'
+          }
+        });
       });
     }
   }, {

package/dist/esm/components/UfoErrorBoundary.js

@@ -28,14 +28,24 @@ export var UfoErrorBoundary = /*#__PURE__*/function (_React$Component) {
 
   _createClass(UfoErrorBoundary, [{
     key: "componentDidCatch",
-    value: function componentDidCatch() {
+    value: function componentDidCatch(error, errorInfo) {
       var _iterator = _createForOfIteratorHelper(this.props.experiences),
           _step;
 
       try {
         for (_iterator.s(); !(_step = _iterator.n()).done;) {
-          var e = _step.value;
-          e.failure();
+          var exp = _step.value;
+          exp.failure({
+            metadata: {
+              source: 'UfoErrorBoundary',
+              reason: 'error',
+              error: {
+                name: error.name,
+                message: error.message,
+                infoStack: errorInfo.componentStack
+              }
+            }
+          });
         }
       } catch (err) {
         _iterator.e(err);

package/dist/esm/containers/ConnectedReactionsView.js

@@ -12,7 +12,8 @@ var ReactionsView = function ReactionsView(props) {
   var experienceInstance = useRef();
   var ari = props.ari,
       createAnalyticsEvent = props.createAnalyticsEvent,
-      store = props.store;
+      store = props.store,
+      containerAri = props.containerAri;
   useEffect(function () {
     experienceInstance.current = ufoExperiences.render.getInstance(ari);
   }, [ari]);
@@ -25,16 +26,23 @@ var ReactionsView = function ReactionsView(props) {
   }, [createAnalyticsEvent, store]); // abort when component gets unmounted
 
   useEffect(function () {
-    return function () {
+    return function cleanup() {
       var _experienceInstance$c;
 
-      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort();
+      (_experienceInstance$c = experienceInstance.current) === null || _experienceInstance$c === void 0 ? void 0 : _experienceInstance$c.abort({
+        metadata: {
+          source: 'Connected-Reactions-View',
+          data: {
+            ari: ari,
+            containerAri: containerAri
+          },
+          reason: 'unmount'
+        }
+      });
     };
-  }, [experienceInstance]);
+  }, [experienceInstance, containerAri, ari]);
 
   var renderChildren = function renderChildren(innerProps) {
-    var containerAri = props.containerAri,
-        ari = props.ari;
     return /*#__PURE__*/React.createElement(FabricElementsAnalyticsContext, {
       data: {
         containerAri: containerAri,

package/dist/esm/store/ReactionsStore.js

@@ -154,7 +154,15 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
 
           exp.failure({
             metadata: {
-              error: error
+              error: error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri: containerAri,
+                ari: ari,
+                emojiId: emojiId,
+                metadata: _this.metadata
+              },
+              reason: 'addReaction fetch failed'
             }
           });
         }
@@ -180,7 +188,15 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
         // ufo add reaction failure
         exp.failure({
           metadata: {
-            error: error
+            error: error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri: containerAri,
+              ari: ari,
+              emojiId: emojiId,
+              metadata: _this.metadata
+            },
+            reason: 'deleteReaction fetch failed'
           }
         });
       });
@@ -195,11 +211,12 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
        * TODO:
        * All reactions are usually fetched in a single call to reactions-service. Need to check why "getReactions" gets called randomly 1-2 times everytime on each fetch request despite using same containerAri.
        */
-      var exp = ufoExperiences.render.getInstance(containerAri); // ufo start reaction experience
+      var exp = ufoExperiences.render.getInstance(containerAri);
+      var arisArr = aris.reduce(utils.flattenAris); // ufo start reaction experience
 
       exp.start();
 
-      _this.client.getReactions(containerAri, aris.reduce(utils.flattenAris)).then(function (value) {
+      _this.client.getReactions(containerAri, arisArr).then(function (value) {
         Object.keys(value).map(function (ari) {
           var reactionsState = _this.getReactionsState(containerAri, ari);
 
@@ -219,7 +236,17 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
             Analytics.createAndFireSafe(_this.createAnalyticsEvent, Analytics.createRestFailedEvent, 'getReactions', error.code);
           }
 
-          exp.failure();
+          exp.failure({
+            metadata: {
+              error: error,
+              source: 'Reaction-Store',
+              data: {
+                containerAri: containerAri,
+                aris: arisArr.join(',')
+              },
+              reason: 'getReactions fetch failed'
+            }
+          });
           return Promise.reject(error);
         }
       });
@@ -242,7 +269,14 @@ export var MemoryReactionsStore = /*#__PURE__*/function () {
         // ufo get reaction details failure
         exp.failure({
           metadata: {
-            error: error
+            error: error,
+            source: 'Reaction-Store',
+            data: {
+              containerAri: containerAri,
+              ari: ari,
+              emojiId: emojiId
+            },
+            reason: 'getDetailedReaction fetch failed'
           }
         });
       });

package/dist/esm/version.json

@@ -1,4 +1,4 @@
 {
 	"name": "@atlaskit/reactions",
-	"version": "21.3.3"
+	"version": "21.3.4"
 }

package/dist/types/components/UfoErrorBoundary.d.ts

@@ -1,8 +1,8 @@
-import React from 'react';
+import React, { ErrorInfo } from 'react';
 import { UFOExperience } from '@atlaskit/ufo';
 export declare class UfoErrorBoundary extends React.Component<{
     experiences: UFOExperience[];
 }> {
-    componentDidCatch(): void;
+    componentDidCatch(error: Error, errorInfo: ErrorInfo): void;
     render(): React.ReactNode;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaskit/reactions",
-  "version": "21.3.3",
+  "version": "21.3.4",
   "description": "Reactions component",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"
@@ -29,7 +29,7 @@
     "@atlaskit/analytics-namespaced-context": "^6.4.0",
     "@atlaskit/analytics-next": "^8.2.0",
     "@atlaskit/button": "^16.3.0",
-    "@atlaskit/ds-lib": "^1.4.0",
+    "@atlaskit/ds-lib": "^2.0.0",
     "@atlaskit/emoji": "^64.7.0",
     "@atlaskit/icon": "^21.10.0",
     "@atlaskit/motion": "^1.1.0",