@atlaskit/profilecard 16.12.1 → 17.1.0

package/dist/es2019/components/User/ProfileCardResourced.js

@@ -1,14 +1,16 @@
 import _extends from "@babel/runtime/helpers/extends";
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import React, { Suspense } from 'react';
+import { withAnalyticsEvents } from '@atlaskit/analytics-next';
 import { GiveKudosLauncherLazy, KudosType } from '@atlaskit/give-kudos';
-import { AnalyticsName } from '../../internal/analytics';
 import filterActions from '../../internal/filterActions';
 import { CardWrapper } from '../../styled/Card';
+import { fireEvent } from '../../util/analytics';
 import { ErrorMessage } from '../Error';
 import ProfileCard from './ProfileCard';
 import UserLoadingState from './UserLoadingState';
-export default class ProfileCardResourced extends React.PureComponent {
+
+class ProfileCardResourced extends React.PureComponent {
   constructor(...args) {
     super(...args);
 
@@ -25,13 +27,14 @@ export default class ProfileCardResourced extends React.PureComponent {
       kudosDrawerOpen: false
     });
 
-    _defineProperty(this, "callAnalytics", (id, options = {}) => {
-      const {
-        analytics
-      } = this.props;
+    _defineProperty(this, "fireAnalytics", payload => {
+      // Don't fire analytics if the component is unmounted
+      if (!this._isMounted) {
+        return;
+      }
 
-      if (analytics) {
-        analytics(id, options);
+      if (this.props.createAnalyticsEvent) {
+        fireEvent(this.props.createAnalyticsEvent, payload);
       }
     });
 
@@ -54,7 +57,7 @@ export default class ProfileCardResourced extends React.PureComponent {
         hasError: false,
         data: null
       }, () => {
-        const requests = Promise.all([this.props.resourceClient.getProfile(cloudId, userId), this.props.resourceClient.getReportingLines(userId), this.props.resourceClient.shouldShowGiveKudos()]);
+        const requests = Promise.all([this.props.resourceClient.getProfile(cloudId, userId, this.fireAnalytics), this.props.resourceClient.getReportingLines(userId), this.props.resourceClient.shouldShowGiveKudos()]);
         requests.then(res => this.handleClientSuccess(...res), err => this.handleClientError(err)).catch(err => this.handleClientError(err));
       });
     });
@@ -85,19 +88,12 @@ export default class ProfileCardResourced extends React.PureComponent {
       cloudId,
       resourceClient
     } = this.props;
-    const {
-      hasError
-    } = this.state;
 
     if (userId !== prevProps.userId || cloudId !== prevProps.cloudId || resourceClient !== prevProps.resourceClient) {
       this.setState({
         isLoading: undefined
       }, this.clientFetchProfile);
     }
-
-    if (hasError !== prevState.hasError && hasError) {
-      this.callAnalytics(AnalyticsName.PROFILE_CARD_RESOURCED_ERROR);
-    }
   }
 
   componentWillUnmount() {
@@ -140,7 +136,6 @@ export default class ProfileCardResourced extends React.PureComponent {
       isKudosEnabled
     } = this.state;
     const {
-      analytics,
       onReportingLinesClick,
       cloudId,
       userId,
@@ -149,11 +144,14 @@ export default class ProfileCardResourced extends React.PureComponent {
     const isFetchingOrNotStartToFetchYet = isLoading === true || isLoading === undefined;
 
     if (isFetchingOrNotStartToFetchYet) {
-      return /*#__PURE__*/React.createElement(CardWrapper, null, /*#__PURE__*/React.createElement(UserLoadingState, null));
+      return /*#__PURE__*/React.createElement(CardWrapper, null, /*#__PURE__*/React.createElement(UserLoadingState, {
+        fireAnalytics: this.fireAnalytics
+      }));
     } else if (hasError) {
       return /*#__PURE__*/React.createElement(CardWrapper, null, /*#__PURE__*/React.createElement(ErrorMessage, {
         errorType: error,
-        reload: this.clientFetchProfile
+        reload: this.clientFetchProfile,
+        fireAnalytics: this.fireAnalytics
       }));
     }
 
@@ -161,7 +159,6 @@ export default class ProfileCardResourced extends React.PureComponent {
       hasError,
       errorType: error,
       clientFetchProfile: this.clientFetchProfile,
-      analytics,
       reportingLines: reportingLinesData,
       onReportingLinesClick: onReportingLinesClick,
       cloudId,
@@ -194,4 +191,7 @@ export default class ProfileCardResourced extends React.PureComponent {
 
 _defineProperty(ProfileCardResourced, "defaultProps", {
   actions: []
-});
+});
+
+export const ProfileCardResourcedInternal = ProfileCardResourced;
+export default withAnalyticsEvents()(ProfileCardResourced);

package/dist/es2019/components/User/ProfileCardTrigger.js

@@ -1,11 +1,13 @@
 import _extends from "@babel/runtime/helpers/extends";
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import React, { Suspense } from 'react';
+import { withAnalyticsEvents } from '@atlaskit/analytics-next';
 import { GiveKudosLauncherLazy, KudosType } from '@atlaskit/give-kudos';
 import Popup from '@atlaskit/popup';
 import { layers } from '@atlaskit/theme/constants';
 import filterActions from '../../internal/filterActions';
 import { CardWrapper } from '../../styled/Card';
+import { cardTriggered, fireEvent } from '../../util/analytics';
 import { DELAY_MS_HIDE, DELAY_MS_SHOW } from '../../util/config';
 import { ProfileCardLazy } from './lazyProfileCard';
 import UserLoadingState from './UserLoadingState';
@@ -24,6 +26,17 @@ class ProfilecardTrigger extends React.PureComponent {
 
     _defineProperty(this, "hideTimer", 0);
 
+    _defineProperty(this, "fireAnalytics", payload => {
+      // Don't fire any analytics if the component is unmounted
+      if (!this._isMounted) {
+        return;
+      }
+
+      if (this.props.createAnalyticsEvent) {
+        fireEvent(this.props.createAnalyticsEvent, payload);
+      }
+    });
+
     _defineProperty(this, "hideProfilecard", () => {
       clearTimeout(this.showTimer);
       clearTimeout(this.hideTimer);
@@ -53,10 +66,22 @@ class ProfilecardTrigger extends React.PureComponent {
       // lozenge in an inline-edit.
       event.stopPropagation();
       this.showProfilecard();
+
+      if (!this.state.visible) {
+        this.fireAnalytics(cardTriggered('user', 'click'));
+      }
+    });
+
+    _defineProperty(this, "onMouseEnter", () => {
+      this.showProfilecard();
+
+      if (!this.state.visible) {
+        this.fireAnalytics(cardTriggered('user', 'hover'));
+      }
     });
 
     _defineProperty(this, "containerListeners", this.props.trigger === 'hover' ? {
-      onMouseEnter: this.showProfilecard,
+      onMouseEnter: this.onMouseEnter,
       onMouseLeave: this.hideProfilecard
     } : {
       onClick: this.onClick
@@ -101,7 +126,7 @@ class ProfilecardTrigger extends React.PureComponent {
         hasError: false,
         data: null
       }, () => {
-        const requests = Promise.all([this.props.resourceClient.getProfile(cloudId || '', userId), this.props.resourceClient.getReportingLines(userId), this.props.resourceClient.shouldShowGiveKudos()]);
+        const requests = Promise.all([this.props.resourceClient.getProfile(cloudId || '', userId, this.fireAnalytics), this.props.resourceClient.getReportingLines(userId), this.props.resourceClient.shouldShowGiveKudos()]);
         requests.then(res => this.handleClientSuccess(...res), err => this.handleClientError(err)).catch(err => this.handleClientError(err));
       });
     });
@@ -125,7 +150,9 @@ class ProfilecardTrigger extends React.PureComponent {
       } = this.state;
 
       if (isLoading === true || isLoading === undefined) {
-        return /*#__PURE__*/React.createElement(CardWrapper, null, /*#__PURE__*/React.createElement(UserLoadingState, null));
+        return /*#__PURE__*/React.createElement(CardWrapper, null, /*#__PURE__*/React.createElement(UserLoadingState, {
+          fireAnalytics: this.fireAnalytics
+        }));
       } else {
         return this.renderProfileCard();
       }
@@ -196,7 +223,6 @@ class ProfilecardTrigger extends React.PureComponent {
       userId: this.props.userId,
       isCurrentUser: (_this$state$data = this.state.data) === null || _this$state$data === void 0 ? void 0 : _this$state$data.isCurrentUser,
       clientFetchProfile: this.clientFetchProfile,
-      analytics: this.props.analytics,
       ...this.state.data,
       reportingLines: this.state.reportingLinesData,
       onReportingLinesClick: this.props.onReportingLinesClick,
@@ -206,7 +232,7 @@ class ProfilecardTrigger extends React.PureComponent {
       openKudosDrawer: this.openKudosDrawer
     };
     const wrapperProps = this.props.trigger === 'hover' ? {
-      onMouseEnter: this.showProfilecard,
+      onMouseEnter: this.onMouseEnter,
       onMouseLeave: this.hideProfilecard
     } : {};
     return /*#__PURE__*/React.createElement("div", wrapperProps, this.state.visible && /*#__PURE__*/React.createElement(Suspense, {
@@ -269,4 +295,4 @@ _defineProperty(ProfilecardTrigger, "defaultProps", {
   position: 'bottom-start'
 });
 
-export default ProfilecardTrigger;
+export default withAnalyticsEvents()(ProfilecardTrigger);

package/dist/es2019/components/User/ReportingLinesDetails.js

@@ -3,9 +3,9 @@ import { FormattedMessage } from 'react-intl-next';
 import Avatar from '@atlaskit/avatar';
 import AvatarGroup from '@atlaskit/avatar-group';
 import Button from '@atlaskit/button/custom-theme-button';
-import { AnalyticsName } from '../../internal/analytics';
 import messages from '../../messages';
 import { ManagerName, ManagerSection, OffsetWrapper, ReportingLinesHeading, ReportingLinesSection } from '../../styled/ReportingLines';
+import { reportingLinesClicked } from '../../util/analytics';
 
 function getProfileHref(userId, profileUrl) {
   return profileUrl ? profileUrl + userId : undefined;
@@ -15,11 +15,10 @@ const ReportingLinesDetails = props => {
   var _manager$pii, _manager$pii2;
 
   const {
+    fireAnalyticsWithDuration,
     reportingLines = {},
     reportingLinesProfileUrl,
-    onReportingLinesClick,
-    analytics = () => {},
-    getDuration
+    onReportingLinesClick
   } = props;
   const {
     managers = [],
@@ -28,11 +27,11 @@ const ReportingLinesDetails = props => {
   const manager = managers.length >= 1 ? managers[0] : undefined;
   const hasReports = reports.length > 0;
 
-  const getReportingLinesOnClick = (user, analyticsId) => onReportingLinesClick ? () => {
-    analytics(AnalyticsName.PROFILE_CARD_CLICK, {
-      id: analyticsId,
-      duration: getDuration()
-    });
+  const getReportingLinesOnClick = (user, userType) => onReportingLinesClick ? () => {
+    fireAnalyticsWithDuration(duration => reportingLinesClicked({
+      duration,
+      userType
+    }));
     onReportingLinesClick(user);
   } : undefined;
 
@@ -44,7 +43,7 @@ const ReportingLinesDetails = props => {
     appearance: "subtle",
     spacing: "none",
     href: getProfileHref(manager.accountIdentifier, reportingLinesProfileUrl),
-    onClick: getReportingLinesOnClick(manager, 'reporting-lines-manager'),
+    onClick: getReportingLinesOnClick(manager, 'manager'),
     isDisabled: !onReportingLinesClick
   }, /*#__PURE__*/React.createElement(ManagerSection, null, /*#__PURE__*/React.createElement(Avatar, {
     size: "xsmall",
@@ -60,7 +59,7 @@ const ReportingLinesDetails = props => {
         name: ((_member$pii = member.pii) === null || _member$pii === void 0 ? void 0 : _member$pii.name) || '',
         src: (_member$pii2 = member.pii) === null || _member$pii2 === void 0 ? void 0 : _member$pii2.picture,
         href: getProfileHref(member.accountIdentifier, reportingLinesProfileUrl),
-        onClick: getReportingLinesOnClick(member, 'reporting-lines-direct-report')
+        onClick: getReportingLinesOnClick(member, 'direct-report')
       };
     }),
     maxCount: 5,

package/dist/es2019/components/User/UserLoadingState.js

@@ -1,7 +1,15 @@
-import React from 'react';
+import React, { useEffect } from 'react';
 import AkSpinner from '@atlaskit/spinner';
 import { SpinnerContainer } from '../../styled/Card';
+import { profileCardRendered } from '../../util/analytics';
 
-const UserLoadingState = () => /*#__PURE__*/React.createElement(SpinnerContainer, null, /*#__PURE__*/React.createElement(AkSpinner, null));
+const UserLoadingState = ({
+  fireAnalytics
+}) => {
+  useEffect(() => {
+    fireAnalytics(profileCardRendered('user', 'spinner'));
+  }, [fireAnalytics]);
+  return /*#__PURE__*/React.createElement(SpinnerContainer, null, /*#__PURE__*/React.createElement(AkSpinner, null));
+};
 
 export default UserLoadingState;

package/dist/es2019/util/analytics.js

@@ -4,8 +4,10 @@ import { getPageTime } from './performance';
 const ANALYTICS_CHANNEL = 'peopleTeams';
 
 const runItLater = cb => {
-  if (window.requestIdleCallback === 'function') {
-    return window.requestIdleCallback(cb);
+  const requestIdleCallback = window.requestIdleCallback;
+
+  if (typeof requestIdleCallback === 'function') {
+    return requestIdleCallback(cb);
   }
 
   if (typeof window.requestAnimationFrame === 'function') {
@@ -27,6 +29,7 @@ export const fireEvent = (createAnalyticsEvent, body) => {
 /** Above lines are copied from teams common analytics */
 
 const TEAM_SUBJECT = 'teamProfileCard';
+const USER_SUBJECT = 'profilecard';
 
 const createEvent = (eventType, action, actionSubject, actionSubjectId, attributes = {}) => ({
   eventType,
@@ -35,19 +38,21 @@ const createEvent = (eventType, action, actionSubject, actionSubjectId, attribut
   actionSubjectId,
   attributes: {
     packageName: "@atlaskit/profilecard",
-    packageVersion: "16.12.1",
+    packageVersion: "17.1.0",
     ...attributes,
-    firedAt: getPageTime()
+    firedAt: Math.round(getPageTime())
   }
 });
 
-export const teamCardTriggered = method => createEvent('ui', 'triggered', TEAM_SUBJECT, undefined, {
+export const cardTriggered = (type, method) => createEvent('ui', 'triggered', type === 'user' ? USER_SUBJECT : TEAM_SUBJECT, undefined, {
   method
 });
 export const teamRequestAnalytics = (action, attributes) => createEvent('operational', action, TEAM_SUBJECT, 'request', attributes);
-export const teamProfileCardRendered = (actionSubjectId, attributes) => createEvent('ui', 'rendered', TEAM_SUBJECT, actionSubjectId, attributes);
-export const teamActionClicked = attributes => createEvent('ui', 'clicked', TEAM_SUBJECT, 'action', attributes);
-export const moreActionsClicked = attributes => createEvent('ui', 'clicked', TEAM_SUBJECT, 'moreActions', attributes);
+export const userRequestAnalytics = (action, attributes) => createEvent('operational', action, USER_SUBJECT, 'request', attributes);
+export const profileCardRendered = (type, actionSubjectId, attributes) => createEvent('ui', 'rendered', type === 'user' ? USER_SUBJECT : TEAM_SUBJECT, actionSubjectId, attributes);
+export const actionClicked = (type, attributes) => createEvent('ui', 'clicked', type === 'user' ? USER_SUBJECT : TEAM_SUBJECT, 'action', attributes);
+export const reportingLinesClicked = attributes => createEvent('ui', 'clicked', USER_SUBJECT, 'reportingLines', attributes);
+export const moreActionsClicked = (type, attributes) => createEvent('ui', 'clicked', type === 'user' ? USER_SUBJECT : TEAM_SUBJECT, 'moreActions', attributes);
 export const teamAvatarClicked = attributes => createEvent('ui', 'clicked', TEAM_SUBJECT, 'avatar', attributes);
 export const moreMembersClicked = attributes => createEvent('ui', 'clicked', TEAM_SUBJECT, 'moreMembers', attributes);
 export const errorRetryClicked = attributes => createEvent('ui', 'clicked', TEAM_SUBJECT, 'errorRetry', attributes);

package/dist/es2019/version.json

@@ -1,4 +1,4 @@
 {
 	"name": "@atlaskit/profilecard",
-	"version": "16.12.1"
+	"version": "17.1.0"
 }

package/dist/esm/client/ProfileCardClient.js

@@ -30,8 +30,8 @@ var ProfileCardClient = /*#__PURE__*/function () {
     }
   }, {
     key: "getProfile",
-    value: function getProfile(cloudId, userId) {
-      return this.userClient.getProfile(cloudId, userId);
+    value: function getProfile(cloudId, userId, analytics) {
+      return this.userClient.getProfile(cloudId, userId, analytics);
     }
   }, {
     key: "getTeamProfile",
@@ -58,11 +58,11 @@ var ProfileCardClient = /*#__PURE__*/function () {
   }, {
     key: "shouldShowGiveKudos",
     value: function shouldShowGiveKudos() {
-      var _this$tcClient4, _this$tcClient5, _this$tcClient6;
+      var _this$tcClient4, _this$tcClient5;
 
       // Check if the kudos feature enabled and if the user has TC
       // if the user does not have TC the tc client will be undefined.
-      return ((_this$tcClient4 = this.tcClient) === null || _this$tcClient4 === void 0 ? void 0 : _this$tcClient4.options.teamCentralBaseUrl) && ((_this$tcClient5 = this.tcClient) === null || _this$tcClient5 === void 0 ? void 0 : _this$tcClient5.getFlagEnabled('team-central-kudos-enabled-m2', (_this$tcClient6 = this.tcClient) === null || _this$tcClient6 === void 0 ? void 0 : _this$tcClient6.options.productIdentifier)) || Promise.resolve(false);
+      return this.getTeamCentralBaseUrl() && ((_this$tcClient4 = this.tcClient) === null || _this$tcClient4 === void 0 ? void 0 : _this$tcClient4.getFlagEnabled('team-central-kudos-enabled-m2', (_this$tcClient5 = this.tcClient) === null || _this$tcClient5 === void 0 ? void 0 : _this$tcClient5.options.productIdentifier)) || Promise.resolve(false);
     }
   }]);
 

package/dist/esm/client/TeamCentralCardClient.js

@@ -32,6 +32,14 @@ var buildCheckFeatureFlagQuery = function buildCheckFeatureFlagQuery(featureKey,
   };
 };
 
+function hasTCWorkspace(config) {
+  return config.cloudId ? fetch("/gateway/api/watermelon/organization/containsAnyWorkspace?cloudId=".concat(config.cloudId)).then(function (res) {
+    return !res || res && res.ok;
+  }) : Promise.resolve(false);
+}
+
+var isTCReadyPromiseMap = new Map();
+
 var TeamCentralCardClient = /*#__PURE__*/function (_CachingClient) {
   _inherits(TeamCentralCardClient, _CachingClient);
 
@@ -54,10 +62,27 @@ var TeamCentralCardClient = /*#__PURE__*/function (_CachingClient) {
     _this.options = options;
     _this.bypassOnFailure = false;
     _this.featureFlagKeys = new Map();
+    _this.isTCReadyPromise = _this.createTcReadyPromise(options);
     return _this;
   }
 
   _createClass(TeamCentralCardClient, [{
+    key: "createTcReadyPromise",
+    value: function createTcReadyPromise(config) {
+      if (config.cloudId) {
+        var promise = isTCReadyPromiseMap.get(config.cloudId);
+
+        if (!promise) {
+          promise = hasTCWorkspace(config);
+          isTCReadyPromiseMap.set(config.cloudId, promise);
+        }
+
+        return promise;
+      }
+
+      return Promise.resolve(true);
+    }
+  }, {
     key: "makeFeatureFlagCheckRequest",
     value: function () {
       var _makeFeatureFlagCheckRequest = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee(featureKey, context) {
@@ -76,7 +101,7 @@ var TeamCentralCardClient = /*#__PURE__*/function (_CachingClient) {
               case 2:
                 query = buildCheckFeatureFlagQuery(featureKey, context);
                 _context.next = 5;
-                return graphqlQuery(this.options.teamCentralUrl, query);
+                return graphqlQuery("".concat(this.options.teamCentralUrl, "?operationName=isFeatureKeyEnabled"), query);
 
               case 5:
                 response = _context.sent;
@@ -115,7 +140,7 @@ var TeamCentralCardClient = /*#__PURE__*/function (_CachingClient) {
               case 2:
                 query = buildReportingLinesQuery(userId);
                 _context2.next = 5;
-                return graphqlQuery(this.options.teamCentralUrl, query);
+                return graphqlQuery("".concat(this.options.teamCentralUrl, "?operationName=ReportingLines"), query);
 
               case 5:
                 response = _context2.sent;
@@ -140,44 +165,58 @@ var TeamCentralCardClient = /*#__PURE__*/function (_CachingClient) {
     value: function getReportingLines(userId) {
       var _this2 = this;
 
-      if (!userId) {
-        return Promise.reject(new Error('userId missing'));
-      }
-
-      var cache = this.getCachedProfile(userId);
-
-      if (cache) {
-        return Promise.resolve(cache);
-      }
-
-      if (this.bypassOnFailure) {
-        return Promise.resolve({});
-      }
+      return this.isTCReadyPromise.then(function (workSpaceExists) {
+        if (workSpaceExists) {
+          if (!userId) {
+            return Promise.reject(new Error('userId missing'));
+          }
 
-      return new Promise(function (resolve) {
-        _this2.makeRequest(userId).then(function (data) {
-          var enhancedData = {
-            managers: _this2.filterReportingLinesUser(data === null || data === void 0 ? void 0 : data.managers),
-            reports: _this2.filterReportingLinesUser(data === null || data === void 0 ? void 0 : data.reports)
-          };
+          var cache = _this2.getCachedProfile(userId);
 
-          if (_this2.cache) {
-            _this2.setCachedProfile(userId, enhancedData);
+          if (cache) {
+            return Promise.resolve(cache);
           }
 
-          resolve(enhancedData);
-        }).catch(function (error) {
-          if ((error === null || error === void 0 ? void 0 : error.status) === 401 || (error === null || error === void 0 ? void 0 : error.status) === 403) {
-            // Trigger circuit breaker
-            _this2.bypassOnFailure = true;
+          if (_this2.bypassOnFailure) {
+            return Promise.resolve({});
           }
-          /**
-           * Reporting lines aren't part of the critical path of profile card.
-           * Just resolve with empty values instead of bubbling up the error.
-           */
-
 
-          resolve({});
+          return new Promise(function (resolve) {
+            _this2.makeRequest(userId).then(function (data) {
+              var enhancedData = {
+                managers: _this2.filterReportingLinesUser(data === null || data === void 0 ? void 0 : data.managers),
+                reports: _this2.filterReportingLinesUser(data === null || data === void 0 ? void 0 : data.reports)
+              };
+
+              if (_this2.cache) {
+                _this2.setCachedProfile(userId, enhancedData);
+              }
+
+              resolve(enhancedData);
+            }).catch(function (error) {
+              if ((error === null || error === void 0 ? void 0 : error.status) === 401 || (error === null || error === void 0 ? void 0 : error.status) === 403) {
+                // Trigger circuit breaker
+                _this2.bypassOnFailure = true;
+              }
+              /**
+               * Reporting lines aren't part of the critical path of profile card.
+               * Just resolve with empty values instead of bubbling up the error.
+               */
+
+
+              resolve({});
+            });
+          });
+        }
+
+        return Promise.resolve({
+          managers: [],
+          reports: []
+        });
+      }, function () {
+        return Promise.resolve({
+          managers: [],
+          reports: []
         });
       });
     }
@@ -186,35 +225,43 @@ var TeamCentralCardClient = /*#__PURE__*/function (_CachingClient) {
     value: function getFlagEnabled(featureKey, productIdentifier) {
       var _this3 = this;
 
-      if (!featureKey) {
-        return Promise.reject(new Error('featureKey missing'));
-      }
-
-      if (this.featureFlagKeys.has(featureKey)) {
-        return Promise.resolve(this.featureFlagKeys.get(featureKey));
-      }
+      return this.isTCReadyPromise.then(function (workSpaceExists) {
+        if (workSpaceExists) {
+          if (!featureKey) {
+            return Promise.reject(new Error('featureKey missing'));
+          }
 
-      if (this.bypassOnFailure) {
-        return Promise.resolve(false);
-      }
+          if (_this3.featureFlagKeys.has(featureKey)) {
+            return Promise.resolve(_this3.featureFlagKeys.get(featureKey));
+          }
 
-      var context = [{
-        key: 'productIdentifier',
-        value: productIdentifier || 'unset'
-      }];
-      return new Promise(function (resolve) {
-        _this3.makeFeatureFlagCheckRequest(featureKey, context).then(function (enabled) {
-          _this3.featureFlagKeys.set(featureKey, enabled);
-
-          resolve(enabled);
-        }).catch(function (error) {
-          if ((error === null || error === void 0 ? void 0 : error.status) === 401 || (error === null || error === void 0 ? void 0 : error.status) === 403) {
-            // Trigger circuit breaker
-            _this3.bypassOnFailure = true;
+          if (_this3.bypassOnFailure) {
+            return Promise.resolve(false);
           }
 
-          resolve(false);
-        });
+          var context = [{
+            key: 'productIdentifier',
+            value: productIdentifier || 'unset'
+          }];
+          return new Promise(function (resolve) {
+            _this3.makeFeatureFlagCheckRequest(featureKey, context).then(function (enabled) {
+              _this3.featureFlagKeys.set(featureKey, enabled);
+
+              resolve(enabled);
+            }).catch(function (error) {
+              if ((error === null || error === void 0 ? void 0 : error.status) === 401 || (error === null || error === void 0 ? void 0 : error.status) === 403) {
+                // Trigger circuit breaker
+                _this3.bypassOnFailure = true;
+              }
+
+              resolve(false);
+            });
+          });
+        }
+
+        return Promise.resolve(false);
+      }, function () {
+        return Promise.resolve(false);
       });
     }
   }, {

package/dist/esm/client/TeamProfileCardClient.js

@@ -1,9 +1,14 @@
+import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import _classCallCheck from "@babel/runtime/helpers/classCallCheck";
 import _createClass from "@babel/runtime/helpers/createClass";
 import _inherits from "@babel/runtime/helpers/inherits";
 import _possibleConstructorReturn from "@babel/runtime/helpers/possibleConstructorReturn";
 import _getPrototypeOf from "@babel/runtime/helpers/getPrototypeOf";
 
+function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
+
+function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { _defineProperty(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
+
 function _createSuper(Derived) { var hasNativeReflectConstruct = _isNativeReflectConstruct(); return function _createSuperInternal() { var Super = _getPrototypeOf(Derived), result; if (hasNativeReflectConstruct) { var NewTarget = _getPrototypeOf(this).constructor; result = Reflect.construct(Super, arguments, NewTarget); } else { result = Super.apply(this, arguments); } return _possibleConstructorReturn(this, result); }; }
 
 function _isNativeReflectConstruct() { if (typeof Reflect === "undefined" || !Reflect.construct) return false; if (Reflect.construct.sham) return false; if (typeof Proxy === "function") return true; try { Boolean.prototype.valueOf.call(Reflect.construct(Boolean, [], function () {})); return true; } catch (e) { return false; } }
@@ -11,6 +16,7 @@ function _isNativeReflectConstruct() { if (typeof Reflect === "undefined" || !Re
 import { teamRequestAnalytics } from '../util/analytics';
 import { getPageTime } from '../util/performance';
 import CachingClient from './CachingClient';
+import { getErrorAttributes } from './errorUtils';
 import { getTeamFromAGG } from './getTeamFromAGG';
 import { graphqlQuery } from './graphqlUtils';
 var QUERY = "query Team($teamId: String!, $organizationId: String) {\n  Team: Team(teamId: $teamId, organizationId: $organizationId) {\n    id,\n    description,\n    displayName,\n    largeHeaderImageUrl,\n    smallHeaderImageUrl,\n    largeAvatarImageUrl,\n    smallAvatarImageUrl,\n    members {\n      id,\n      fullName,\n      avatarUrl,\n    },\n  }\n}";
@@ -25,12 +31,6 @@ var buildTeamQuery = function buildTeamQuery(teamId, orgId) {
   };
 };
 
-var IGNORED_ERRORS = ['NotPermitted', 'Gone'];
-
-function isRealError(error) {
-  return !IGNORED_ERRORS.includes(error.reason);
-}
-
 var TeamProfileCardClient = /*#__PURE__*/function (_CachingClient) {
   _inherits(TeamProfileCardClient, _CachingClient);
 
@@ -105,13 +105,12 @@ var TeamProfileCardClient = /*#__PURE__*/function (_CachingClient) {
 
           resolve(data);
         }).catch(function (error) {
-          if (analytics && isRealError(error)) {
-            analytics(teamRequestAnalytics('failed', {
-              duration: getPageTime() - startTime,
-              errorStatus: error.code,
-              errorReason: error.reason,
+          if (analytics) {
+            analytics(teamRequestAnalytics('failed', _objectSpread(_objectSpread({
+              duration: getPageTime() - startTime
+            }, getErrorAttributes(error)), {}, {
               gateway: shouldUseGateway
-            }));
+            })));
           }
 
           reject(error);