change post-retro DM to simply link to web UI

Author: jeffreywescott

server/actions/sendPlayerStatsSummaries.js

@@ -0,0 +1,43 @@
+import config from 'src/config'
+import getPlayerInfo from 'src/server/actions/getPlayerInfo'
+import {findPlayersByIds} from 'src/server/db/player'
+
+export default async function sendRetroCompletedNotification(project) {
+  const chatService = require('src/server/services/chatService')
+
+  const projectPlayers = await findPlayersByIds(project.playerIds)
+  const projectPlayerUsers = await getPlayerInfo(project.playerIds)
+  const players = _mergePlayerUsers(projectPlayers, projectPlayerUsers)
+
+  return Promise.all(players.map(player => {
+    const retroStatsMessage = _compilePlayerStatsMessage(player, project)
+
+    return chatService.sendDirectMessage(player.handle, retroStatsMessage).catch(err => {
+      console.error(`\n\nThere was a problem while sending stats DM to player @${player.handle}`)
+      console.error('Error:', err, err.stack)
+      console.error(`Message: "${retroStatsMessage}"`)
+    })
+  }))
+}
+
+function _mergePlayerUsers(players, users) {
+  const combined = new Map()
+
+  players.forEach(player => combined.set(player.id, Object.assign({}, player)))
+
+  users.forEach(user => {
+    const values = combined.get(user.id)
+    if (values) {
+      combined.set(user.id, Object.assign({}, values, user))
+    }
+  })
+
+  return Array.from(combined.values())
+}
+
+function _compilePlayerStatsMessage(player, project) {
+  return `**RETROSPECTIVE COMPLETE:**
+
+  - [View Project: ${project.name}](${config.app.baseURL}/projects/${project.name})
+  - [View Your Stats](${config.app.baseURL}/users/${player.handle})`
+}

server/actions/sendRetroCompletedNotification.js

@@ -91,7 +91,7 @@ describe(testContext(__filename), function () {
           await processSurveyResponseSubmitted(event)
           expect(chatService.sendDirectMessage.callCount).to.eq(this.users.length)
           this.users.forEach(user => (
-            expect(chatService.sendDirectMessage).to.have.been.calledWithMatch(user.handle, 'RETROSPECTIVE RESULTS')
+            expect(chatService.sendDirectMessage).to.have.been.calledWithMatch(user.handle, 'RETROSPECTIVE COMPLETE')
           ))
         })
       })

server/workers/__tests__/surveyResponseSubmitted.test.js

@@ -2,7 +2,7 @@ import Promise from 'bluebird'
 
 import {findProjectBySurveyId} from 'src/server/db/project'
 import {getSurveyById, recordSurveyCompletedBy, surveyWasCompletedBy} from 'src/server/db/survey'
-import sendPlayerStatsSummaries from 'src/server/actions/sendPlayerStatsSummaries'
+import sendRetroCompletedNotification from 'src/server/actions/sendRetroCompletedNotification'
 import updatePlayerStatsForProject from 'src/server/actions/updatePlayerStatsForProject'
 import updateProjectStats from 'src/server/actions/updateProjectStats'
 import updatePlayerCumulativeStats from 'src/server/actions/updatePlayerCumulativeStats'
@@ -84,7 +84,7 @@ async function updateStatsIfNeeded(project, survey, respondentId) {
     console.log(`All respondents have completed this survey [${survey.id}]. Updating stats.`)
     await updateProjectStats(project.id)
     await updatePlayerStatsForProject(project)
-    await sendPlayerStatsSummaries(project)
+    await sendRetroCompletedNotification(project)
   }
 }