@atlaskit/smart-card 16.1.1 → 16.2.0

package/dist/es2019/view/CardWithUrl/loader.js

@@ -1,60 +1,32 @@
-import _defineProperty from "@babel/runtime/helpers/defineProperty";
-import React from 'react';
+import React, { lazy, useEffect, useState, Suspense } from 'react';
 import uuid from 'uuid';
 import { CardLinkView } from '@atlaskit/media-ui';
 import { uiRenderFailedEvent, fireSmartLinkEvent } from '../../utils/analytics';
 import { clearMarks, clearMeasures } from '../../utils/performance';
-export class CardWithURLRenderer extends React.PureComponent {
-  constructor(props) {
-    super(props);
+import { ErrorBoundary } from 'react-error-boundary';
+import { failUfoExperience, startUfoExperience } from '../../state/analytics/ufoExperiences';
+const LazyCardWithUrlContent = /*#__PURE__*/lazy(() => import(
+/* webpackChunkName: "@atlaskit-internal_smartcard-urlcardcontent" */
+'./component-lazy/index'));
+export function CardWithURLRenderer(props) {
+  const [id] = useState(() => uuid()); // Equivalent to ComponentWillMount
 
-    _defineProperty(this, "dispatchAnalytics", analyticsPayload => {
-      const {
-        appearance,
-        createAnalyticsEvent
-      } = this.props;
-
-      if (analyticsPayload && analyticsPayload.attributes) {
-        // Update if we haven't already set the display - possible
-        // in the case of `preview` which is rendered differently.
-        if (!analyticsPayload.attributes.display) {
-          analyticsPayload.attributes.display = appearance;
-        }
-      }
-
-      fireSmartLinkEvent(analyticsPayload, createAnalyticsEvent);
-    });
-
-    this.state = {
-      id: uuid()
+  useState(() => {
+    // We want to start timing of render event only once and right at the start
+    startUfoExperience('smart-link-rendered', id);
+  });
+  useEffect(() => {
+    // ComponentWillUnmount
+    return () => {
+      clearMarks(id);
+      clearMeasures(id);
     };
-  }
-
-  static moduleImporter(target) {
-    import(
-    /* webpackChunkName: "@atlaskit-internal_smartcard-urlcardcontent" */
-    './component-lazy/index').then(module => {
-      CardWithURLRenderer.CardContent = module.LazyCardWithUrlContent;
-      target.forceUpdate();
-    });
-  }
-
-  componentDidMount() {
-    if (CardWithURLRenderer.CardContent === null) {
-      (this.props.importer || CardWithURLRenderer.moduleImporter)(this);
-    }
-  }
+  }, [id]);
 
-  componentWillUnmount() {
-    const id = this.state.id;
-    clearMarks(id);
-    clearMeasures(id);
-  }
-
-  componentDidCatch(error, errorInfo) {
+  const logError = (error, info) => {
     const {
-      appearance
-    } = this.props; // NB: APIErrors are thrown in response to Object Resolver Service.
+      componentStack
+    } = info; // NB: APIErrors are thrown in response to Object Resolver Service.
     // In these cases, control is handed back to the Editor. We do not
     // fire an event for these, as they do not cover failed UI render events.
 
@@ -64,54 +36,80 @@ export class CardWithURLRenderer extends React.PureComponent {
     // to the reliability of the smart-card front-end components. We instrument
     // these in order to measure our front-end reliability.
     else {
-      this.dispatchAnalytics(uiRenderFailedEvent(appearance, error, errorInfo));
+      const errorInfo = {
+        componentStack
+      };
+      failUfoExperience('smart-link-rendered', id);
+      failUfoExperience('smart-link-authenticated', id);
+      dispatchAnalytics(uiRenderFailedEvent(appearance, error, errorInfo));
     }
-  } // Wrapper around analytics.
+  }; // Wrapper around analytics.
 
 
-  render() {
+  const dispatchAnalytics = analyticsPayload => {
     const {
-      url,
       appearance,
-      isSelected,
-      isFrameVisible,
-      onClick,
-      container,
-      onResolve,
-      testId,
-      showActions,
-      inheritDimensions,
-      platform,
-      embedIframeRef,
-      inlinePreloaderStyle
-    } = this.props;
+      createAnalyticsEvent
+    } = props;
 
-    if (!url) {
-      throw new Error('@atlaskit/smart-card: url property is missing.');
+    if (analyticsPayload && analyticsPayload.attributes) {
+      // Update if we haven't already set the display - possible
+      // in the case of `preview` which is rendered differently.
+      if (!analyticsPayload.attributes.display) {
+        analyticsPayload.attributes.display = appearance;
+      }
     }
 
-    return CardWithURLRenderer.CardContent !== null ? /*#__PURE__*/React.createElement(CardWithURLRenderer.CardContent, {
-      id: this.state.id,
-      url: url,
-      appearance: appearance,
-      onClick: onClick,
-      isSelected: isSelected,
-      isFrameVisible: isFrameVisible,
-      dispatchAnalytics: this.dispatchAnalytics,
-      container: container,
-      onResolve: onResolve,
-      testId: testId,
-      showActions: showActions,
-      inheritDimensions: inheritDimensions,
-      platform: platform,
-      embedIframeRef: embedIframeRef,
-      inlinePreloaderStyle: inlinePreloaderStyle
-    }) : /*#__PURE__*/React.createElement(CardLinkView, {
-      key: 'chunk-placeholder',
-      link: url
-    });
+    fireSmartLinkEvent(analyticsPayload, createAnalyticsEvent);
+  };
+
+  const {
+    url,
+    appearance,
+    isSelected,
+    isFrameVisible,
+    onClick,
+    container,
+    onResolve,
+    testId,
+    showActions,
+    inheritDimensions,
+    platform,
+    embedIframeRef,
+    inlinePreloaderStyle
+  } = props;
+
+  if (!url) {
+    throw new Error('@atlaskit/smart-card: url property is missing.');
   }
 
+  return /*#__PURE__*/React.createElement(ErrorBoundary, {
+    FallbackComponent: ErrorFallback,
+    onError: logError
+  }, /*#__PURE__*/React.createElement(Suspense, {
+    fallback: /*#__PURE__*/React.createElement(CardLinkView, {
+      key: 'chunk-placeholder',
+      link: url
+    })
+  }, /*#__PURE__*/React.createElement(LazyCardWithUrlContent, {
+    id: id,
+    url: url,
+    appearance: appearance,
+    onClick: onClick,
+    isSelected: isSelected,
+    isFrameVisible: isFrameVisible,
+    dispatchAnalytics: dispatchAnalytics,
+    container: container,
+    onResolve: onResolve,
+    testId: testId,
+    showActions: showActions,
+    inheritDimensions: inheritDimensions,
+    platform: platform,
+    embedIframeRef: embedIframeRef,
+    inlinePreloaderStyle: inlinePreloaderStyle
+  })));
 }
 
-_defineProperty(CardWithURLRenderer, "CardContent", null);
+const ErrorFallback = () => {
+  return /*#__PURE__*/React.createElement("span", null);
+};

package/dist/esm/client/utils/environments.js

@@ -1,5 +1,5 @@
 var devBaseUrl = 'https://api-private.dev.atlassian.com';
-var stgBaseUrl = 'https://api-private.stg.atlassian.com';
+var stgBaseUrl = 'https://commerce-components-preview.dev.atlassian.com/gateway/api';
 var prodBaseUrl = 'https://api-private.atlassian.com';
 export var BaseUrls = {
   dev: devBaseUrl,

package/dist/esm/providers/editor/transformer.js

@@ -6,7 +6,7 @@ var isJiraRoadMap = function isJiraRoadMap(url) {
 };
 
 var isPolarisView = function isPolarisView(url) {
-  return url.match(/^https:\/\/.*?\/jira\/polaris\/projects\/[^\/]+?\/ideas\/view\/\d+$/);
+  return url.match(/^https:\/\/.*?\/jira\/polaris\/projects\/[^\/]+?\/ideas\/view\/\d+$|^https:\/\/.*?\/secure\/JiraProductDiscoveryAnonymous\.jspa\?hash=\w+|^https:\/\/.*?\/jira\/polaris\/share\/\w+/);
 };
 
 var isSlackMessage = function isSlackMessage(url) {

package/dist/esm/state/actions/index.js

@@ -9,6 +9,7 @@ import { useSmartLinkContext } from '../context';
 import * as measure from '../../utils/performance';
 import { APIError } from '../../client/errors';
 import { getUnauthorizedJsonLd } from '../../utils/jsonld';
+import { addMetadataToExperience } from '../analytics';
 export var useSmartCardActions = function useSmartCardActions(id, url, analytics) {
   var _useSmartLinkContext = useSmartLinkContext(),
       store = _useSmartLinkContext.store,
@@ -98,7 +99,7 @@ export var useSmartCardActions = function useSmartCardActions(id, url, analytics
             isReloading = _args.length > 1 && _args[1] !== undefined ? _args[1] : false;
 
             if (!(isReloading || !hasData)) {
-              _context.next = 4;
+              _context.next = 6;
               break;
             }
 
@@ -108,13 +109,18 @@ export var useSmartCardActions = function useSmartCardActions(id, url, analytics
               return handleResolvedLinkError(resourceUrl, error);
             }));
 
-          case 4:
+          case 6:
+            addMetadataToExperience('smart-link-rendered', id, {
+              cached: true
+            });
+
+          case 7:
           case "end":
             return _context.stop();
         }
       }
     }, _callee);
-  })), [url, hasData, connections.client, handleResolvedLinkResponse, handleResolvedLinkError]);
+  })), [url, hasData, connections.client, handleResolvedLinkResponse, handleResolvedLinkError, id]);
   var register = useCallback(function () {
     if (!details) {
       dispatch(cardAction(ACTION_PENDING, {
@@ -152,10 +158,10 @@ export var useSmartCardActions = function useSmartCardActions(id, url, analytics
       analytics.screen.authPopupEvent(definitionId, extensionKey);
       auth(services[0].url).then(function () {
         analytics.track.appAccountConnected(definitionId, extensionKey);
-        analytics.operational.connectSucceededEvent(definitionId, extensionKey);
+        analytics.operational.connectSucceededEvent(id, definitionId, extensionKey);
         reload();
       }, function (err) {
-        analytics.operational.connectFailedEvent(definitionId, extensionKey, err.type);
+        analytics.operational.connectFailedEvent(id, definitionId, extensionKey, err.type);
 
         if (err.type === 'auth_window_closed') {
           analytics.ui.closedAuthEvent(appearance, definitionId, extensionKey);
@@ -164,7 +170,7 @@ export var useSmartCardActions = function useSmartCardActions(id, url, analytics
         reload();
       });
     }
-  }, [analytics.ui, analytics.screen, analytics.track, analytics.operational, reload, details, status]);
+  }, [analytics.ui, analytics.screen, analytics.track, analytics.operational, id, reload, details, status]);
   var invoke = useCallback( /*#__PURE__*/function () {
     var _ref3 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2(opts, appearance) {
       var key, action, source, markName, response;

package/dist/esm/state/analytics/index.js

@@ -1,80 +1,2 @@
-import { uiAuthEvent, uiAuthAlternateAccountEvent, uiCardClickedEvent, uiActionClickedEvent, uiClosedAuthEvent, uiRenderSuccessEvent, uiRenderFailedEvent, resolvedEvent as _resolvedEvent, unresolvedEvent as _unresolvedEvent, invokeSucceededEvent as _invokeSucceededEvent, invokeFailedEvent as _invokeFailedEvent, connectSucceededEvent as _connectSucceededEvent, connectFailedEvent as _connectFailedEvent, trackAppAccountConnected, screenAuthPopupEvent, instrumentEvent } from '../../utils/analytics';
-import { useMemo } from 'react';
-export var useSmartLinkAnalytics = function useSmartLinkAnalytics(dispatchAnalytics) {
-  var ui = useMemo(function () {
-    return {
-      authEvent: function authEvent(display, definitionId, extensionKey) {
-        return dispatchAnalytics(uiAuthEvent(display, definitionId, extensionKey));
-      },
-      authAlternateAccountEvent: function authAlternateAccountEvent(display, definitionId, extensionKey) {
-        return dispatchAnalytics(uiAuthAlternateAccountEvent(display, definitionId, extensionKey));
-      },
-      cardClickedEvent: function cardClickedEvent(display, definitionId, extensionKey) {
-        return dispatchAnalytics(uiCardClickedEvent(display, definitionId, extensionKey));
-      },
-      actionClickedEvent: function actionClickedEvent(providerKey, actionType, display) {
-        return dispatchAnalytics(uiActionClickedEvent(providerKey, actionType, display));
-      },
-      closedAuthEvent: function closedAuthEvent(display, definitionId, extensionKey) {
-        return dispatchAnalytics(uiClosedAuthEvent(display, definitionId, extensionKey));
-      },
-      renderSuccessEvent: function renderSuccessEvent(display, definitionId, extensionKey) {
-        return dispatchAnalytics(uiRenderSuccessEvent(display, definitionId, extensionKey));
-      },
-      renderFailedEvent: function renderFailedEvent(display, error, errorInfo) {
-        return dispatchAnalytics(uiRenderFailedEvent(display, error, errorInfo));
-      }
-    };
-  }, [dispatchAnalytics]);
-  var operational = useMemo(function () {
-    return {
-      resolvedEvent: function resolvedEvent(id, definitionId, extensionKey, resourceType) {
-        return dispatchAnalytics(_resolvedEvent(id, definitionId, extensionKey, resourceType));
-      },
-      unresolvedEvent: function unresolvedEvent(id, status, definitionId, extensionKey, resourceType) {
-        return dispatchAnalytics(_unresolvedEvent(id, status, definitionId, extensionKey, resourceType));
-      },
-      invokeSucceededEvent: function invokeSucceededEvent(id, providerKey, actionType, display) {
-        return dispatchAnalytics(_invokeSucceededEvent(id, providerKey, actionType, display));
-      },
-      invokeFailedEvent: function invokeFailedEvent(id, providerKey, actionType, display, reason) {
-        return dispatchAnalytics(_invokeFailedEvent(id, providerKey, actionType, display, reason));
-      },
-      connectSucceededEvent: function connectSucceededEvent(definitionId, extensionKey) {
-        return dispatchAnalytics(_connectSucceededEvent(definitionId, extensionKey));
-      },
-      connectFailedEvent: function connectFailedEvent(definitionId, extensionKey, reason) {
-        return dispatchAnalytics(_connectFailedEvent(definitionId, extensionKey, reason));
-      },
-      instrument: function instrument(id, status, definitionId, extensionKey, resourceType, error) {
-        var event = instrumentEvent(id, status, definitionId, extensionKey, resourceType, error);
-
-        if (event) {
-          dispatchAnalytics(event);
-        }
-      }
-    };
-  }, [dispatchAnalytics]);
-  var track = useMemo(function () {
-    return {
-      appAccountConnected: function appAccountConnected(definitionId, extensionKey) {
-        return dispatchAnalytics(trackAppAccountConnected(definitionId, extensionKey));
-      }
-    };
-  }, [dispatchAnalytics]);
-  var screen = useMemo(function () {
-    return {
-      authPopupEvent: function authPopupEvent(definitionId, extensionKey) {
-        return dispatchAnalytics(screenAuthPopupEvent(definitionId, extensionKey));
-      }
-    };
-  }, [dispatchAnalytics]);
-  return useMemo(function () {
-    return {
-      ui: ui,
-      operational: operational,
-      track: track,
-      screen: screen
-    };
-  }, [ui, operational, track, screen]);
-};
+export { startUfoExperience, succeedUfoExperience, failUfoExperience, addMetadataToExperience } from './ufoExperiences';
+export { useSmartLinkAnalytics } from './useSmartLinkAnalytics';

package/dist/esm/state/analytics/ufoExperiences.js

@@ -0,0 +1,40 @@
+import { ConcurrentExperience, ExperiencePerformanceTypes, ExperienceTypes } from '@atlaskit/ufo';
+var ufoExperiences = {
+  'smart-link-rendered': new ConcurrentExperience('smart-link-rendered', {
+    type: ExperienceTypes.Load,
+    performanceType: ExperiencePerformanceTypes.PageSegmentLoad,
+    platform: {
+      component: 'smart-links'
+    }
+  }),
+  'smart-link-authenticated': new ConcurrentExperience('smart-link-authenticated', {
+    type: ExperienceTypes.Load,
+    performanceType: ExperiencePerformanceTypes.PageSegmentLoad,
+    platform: {
+      component: 'smart-links'
+    }
+  })
+};
+export var startUfoExperience = function startUfoExperience(experienceName, id, properties) {
+  var experience = ufoExperiences[experienceName].getInstance(id);
+  experience.start();
+
+  if (properties) {
+    experience.addMetadata(properties);
+  }
+};
+export var succeedUfoExperience = function succeedUfoExperience(experienceName, id, properties) {
+  var experience = ufoExperiences[experienceName].getInstance(id);
+  experience.success({
+    metadata: properties
+  });
+};
+export var failUfoExperience = function failUfoExperience(experienceName, id, properties) {
+  var experience = ufoExperiences[experienceName].getInstance(id);
+  experience.failure({
+    metadata: properties
+  });
+};
+export var addMetadataToExperience = function addMetadataToExperience(experienceName, id, properties) {
+  ufoExperiences[experienceName].getInstance(id).addMetadata(properties);
+};

package/dist/esm/state/analytics/useSmartLinkAnalytics.js

@@ -0,0 +1,94 @@
+import { uiAuthEvent, uiAuthAlternateAccountEvent, uiCardClickedEvent, uiActionClickedEvent, uiClosedAuthEvent, uiRenderSuccessEvent, resolvedEvent as _resolvedEvent, unresolvedEvent as _unresolvedEvent, invokeSucceededEvent as _invokeSucceededEvent, invokeFailedEvent as _invokeFailedEvent, connectSucceededEvent as _connectSucceededEvent, connectFailedEvent as _connectFailedEvent, trackAppAccountConnected, screenAuthPopupEvent, instrumentEvent } from '../../utils/analytics';
+import { useMemo } from 'react';
+import { startUfoExperience, succeedUfoExperience } from './ufoExperiences';
+export var useSmartLinkAnalytics = function useSmartLinkAnalytics(dispatchAnalytics) {
+  var ui = useMemo(function () {
+    return {
+      authEvent: function authEvent(display, definitionId, extensionKey) {
+        return dispatchAnalytics(uiAuthEvent(display, definitionId, extensionKey));
+      },
+      authAlternateAccountEvent: function authAlternateAccountEvent(display, definitionId, extensionKey) {
+        return dispatchAnalytics(uiAuthAlternateAccountEvent(display, definitionId, extensionKey));
+      },
+      cardClickedEvent: function cardClickedEvent(display, definitionId, extensionKey) {
+        return dispatchAnalytics(uiCardClickedEvent(display, definitionId, extensionKey));
+      },
+      actionClickedEvent: function actionClickedEvent(providerKey, actionType, display) {
+        return dispatchAnalytics(uiActionClickedEvent(providerKey, actionType, display));
+      },
+      closedAuthEvent: function closedAuthEvent(display, definitionId, extensionKey) {
+        return dispatchAnalytics(uiClosedAuthEvent(display, definitionId, extensionKey));
+      },
+      renderSuccessEvent: function renderSuccessEvent(display, id, definitionId, extensionKey) {
+        succeedUfoExperience('smart-link-rendered', id, {
+          extensionKey: extensionKey,
+          display: display
+        }); // UFO will disregard this if authentication experience has not yet been started
+
+        succeedUfoExperience('smart-link-authenticated', id, {
+          display: display
+        });
+        dispatchAnalytics(uiRenderSuccessEvent(display, definitionId, extensionKey));
+      }
+    };
+  }, [dispatchAnalytics]);
+  var operational = useMemo(function () {
+    return {
+      resolvedEvent: function resolvedEvent(id, definitionId, extensionKey, resourceType) {
+        return dispatchAnalytics(_resolvedEvent(id, definitionId, extensionKey, resourceType));
+      },
+      unresolvedEvent: function unresolvedEvent(id, status, definitionId, extensionKey, resourceType) {
+        return dispatchAnalytics(_unresolvedEvent(id, status, definitionId, extensionKey, resourceType));
+      },
+      invokeSucceededEvent: function invokeSucceededEvent(id, providerKey, actionType, display) {
+        return dispatchAnalytics(_invokeSucceededEvent(id, providerKey, actionType, display));
+      },
+      invokeFailedEvent: function invokeFailedEvent(id, providerKey, actionType, display, reason) {
+        return dispatchAnalytics(_invokeFailedEvent(id, providerKey, actionType, display, reason));
+      },
+      connectSucceededEvent: function connectSucceededEvent(id, definitionId, extensionKey) {
+        startUfoExperience('smart-link-authenticated', id, {
+          extensionKey: extensionKey,
+          status: 'success'
+        });
+        dispatchAnalytics(_connectSucceededEvent(definitionId, extensionKey));
+      },
+      connectFailedEvent: function connectFailedEvent(id, definitionId, extensionKey, status) {
+        startUfoExperience('smart-link-authenticated', id, {
+          extensionKey: extensionKey,
+          status: status
+        });
+        dispatchAnalytics(_connectFailedEvent(definitionId, extensionKey, status));
+      },
+      instrument: function instrument(id, status, definitionId, extensionKey, resourceType, error) {
+        var event = instrumentEvent(id, status, definitionId, extensionKey, resourceType, error);
+
+        if (event) {
+          dispatchAnalytics(event);
+        }
+      }
+    };
+  }, [dispatchAnalytics]);
+  var track = useMemo(function () {
+    return {
+      appAccountConnected: function appAccountConnected(definitionId, extensionKey) {
+        return dispatchAnalytics(trackAppAccountConnected(definitionId, extensionKey));
+      }
+    };
+  }, [dispatchAnalytics]);
+  var screen = useMemo(function () {
+    return {
+      authPopupEvent: function authPopupEvent(definitionId, extensionKey) {
+        return dispatchAnalytics(screenAuthPopupEvent(definitionId, extensionKey));
+      }
+    };
+  }, [dispatchAnalytics]);
+  return useMemo(function () {
+    return {
+      ui: ui,
+      operational: operational,
+      track: track,
+      screen: screen
+    };
+  }, [ui, operational, track, screen]);
+};

package/dist/esm/state/hooks/useSmartLink.js

@@ -12,16 +12,15 @@ export function useSmartLink(id, url, dispatchAnalytics) {
   var config = useSmartLinkConfig();
   var renderers = useSmartLinkRenderers(); // NB: used to propagate errors from hooks to error boundaries.
 
-  var _useState = useState(),
+  var _useState = useState(null),
       _useState2 = _slicedToArray(_useState, 2),
-      setState = _useState2[1]; // Register the current card.
+      error = _useState2[0],
+      setError = _useState2[1]; // Register the current card.
 
 
   var register = function register() {
     actions.register().catch(function (err) {
-      return setState(function () {
-        throw err;
-      });
+      return setError(err);
     });
   }; // AFP-2511 TODO: Fix automatic suppressions below
   // eslint-disable-next-line react-hooks/exhaustive-deps
@@ -34,6 +33,7 @@ export function useSmartLink(id, url, dispatchAnalytics) {
     actions: actions,
     config: config,
     analytics: analytics,
-    renderers: renderers
+    renderers: renderers,
+    error: error
   };
 }

package/dist/esm/version.json

@@ -1,5 +1,5 @@
 {
 	"name": "@atlaskit/smart-card",
-	"version": "16.1.1",
+	"version": "16.2.0",
 	"sideEffects": false
 }

package/dist/esm/view/CardWithUrl/component-lazy/index.js

@@ -2,7 +2,7 @@ import React from 'react';
 import { isIntersectionObserverSupported } from '@atlaskit/media-ui';
 import { LazyLazilyRenderCard } from './LazyLazilyRenderCard';
 import { LazyIntersectionObserverCard } from './LazyIntersectionObserverCard';
-export function LazyCardWithUrlContent(props) {
+export default function LazyCardWithUrlContent(props) {
   if (isIntersectionObserverSupported()) {
     return /*#__PURE__*/React.createElement(LazyIntersectionObserverCard, props);
   } else {

package/dist/esm/view/CardWithUrl/component.js

@@ -28,7 +28,8 @@ export function CardWithUrlContent(_ref) {
       actions = _useSmartLink.actions,
       config = _useSmartLink.config,
       analytics = _useSmartLink.analytics,
-      renderers = _useSmartLink.renderers;
+      renderers = _useSmartLink.renderers,
+      error = _useSmartLink.error;
 
   var definitionId = getDefinitionId(state.details);
   var extensionKey = getExtensionKey(state.details);
@@ -71,9 +72,13 @@ export function CardWithUrlContent(_ref) {
 
   useEffect(function () {
     if (isFinalState(state.status)) {
-      analytics.ui.renderSuccessEvent(appearance, definitionId, extensionKey);
+      analytics.ui.renderSuccessEvent(appearance, id, definitionId, extensionKey);
     }
-  }, [appearance, state.details, state.status, url, definitionId, extensionKey, analytics.ui]);
+  }, [appearance, state.details, state.status, url, definitionId, extensionKey, analytics.ui, id]); // We have to keep this last to prevent hook order from being violated
+
+  if (error) {
+    throw error;
+  }
 
   switch (appearance) {
     case 'inline':