Added trackError parameter for presence middleware

foxford · Oct 11, 2023 · fd708dd · fd708dd
1 parent bfc811e
commit fd708dd
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 8 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@ulms/api-clients",
-  "version": "6.6.1",
+  "version": "6.7.0",
   "description": "JavaScript API clients for ULMS platform",
   "keywords": [],
   "homepage": "https://github.com/foxford/ulms-api-clients-js#readme",

diff --git a/src/redux/middleware.js b/src/redux/middleware.js
@@ -57,7 +57,7 @@ const messageHandler = ({ dispatch }, message) => {
 
 async function startPresenceFlow(
   { dispatch },
-  { presenceWs, trackEvent },
+  { presenceWs, trackError, trackEvent },
   classroomId,
   agentLabel = 'http'
 ) {
@@ -134,6 +134,10 @@ async function startPresenceFlow(
         ? { message: error.message, name: error.name }
         : null
 
+      if (trackError) {
+        trackError(error)
+      }
+
       if (trackEvent) {
         trackEvent('Debug', 'Presence.Error', 'v1', undefined, {
           error: errorPayload,
@@ -172,7 +176,7 @@ async function startPresenceFlow(
 
 async function getPresenceAgentList(
   { dispatch, getState },
-  { presence },
+  { presence, trackError },
   classroomId
 ) {
   const agentBackoff = new Backoff()
@@ -220,6 +224,10 @@ async function getPresenceAgentList(
     } catch (error) {
       debug('[agent] catch', error)
 
+      if (trackError) {
+        trackError(error)
+      }
+
       const { status } = error
       const isErrorUnrecoverable = !!status
       const retryLimitExceeded = retryCount === RETRY_LIMIT
@@ -332,7 +340,7 @@ async function getPresenceAgentList(
 }
 
 const createPresenceMiddleware =
-  ({ presence, presenceWs, trackEvent }) =>
+  ({ presence, presenceWs, trackError, trackEvent }) =>
   ({ getState, dispatch }) => {
     const boundedMessageHandler = messageHandler.bind(undefined, {
       dispatch,
@@ -344,15 +352,15 @@ const createPresenceMiddleware =
         dispatch,
         getState,
       },
-      { presence, presenceWs, trackEvent }
+      { presence, presenceWs, trackError, trackEvent }
     )
     const boundedGetPresenceAgentList = getPresenceAgentList.bind(
       undefined,
       {
         dispatch,
         getState,
       },
-      { presence, presenceWs }
+      { presence, presenceWs, trackError, trackEvent }
     )
 
     presenceWs.on('event', boundedMessageHandler)