Hide suggested actions not destinated to the user

CHANGELOG.md

@@ -68,6 +68,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - `bundle`: Fix [#1652](https://github.com/Microsoft/BotFramework-WebChat/issues/1652). Pass `pollingInterval` to DirectLineJS constructor, by [@neetu-das](https://github.com/neetu-das) in PR [#1655](https://github.com/Microsoft/BotFramework-WebChat/pull/1655)
 - `core`: Reworked logic on connect/disconnect for reliability on handling corner cases, by [@compulim](https://github.com/compulim) in PR [#1649](https://github.com/Microsoft/BotFramework-WebChat/pull/1649)
 - `core`: Fix [#1521](https://github.com/Microsoft/BotFramework-WebChat/issues/1521). Add connectivity status component and update localization, by [@corinagum](https://github.com/corinagum) in PR [#1679](https://github.com/Microsoft/BotFramework-WebChat/pull/1679)
+- Fix [#1057](https://github.com/Microsoft/BotFramework-WebChat/issues/1057). Fixed suggested actions destinated for other recipients should not show up, by [@compulim](https://github.com/compulim) in PR [#1706](https://github.com/Microsoft/BotFramework-WebChat/pull/1706)
 
 ### Removed
 - `botAvatarImage` and `userAvatarImage` props, as they are moved inside `styleOptions`, in PR [#1486](https://github.com/Microsoft/BotFramework-WebChat/pull/1486)

__tests__/suggestedActions.js

@@ -148,4 +148,22 @@ describe('suggested-actions command', async () => {
 
     expect(base64PNG).toMatchImageSnapshot(imageSnapshotOptions);
   }, 60000);
+
+  test('should not show suggested actions not destinated for the user', async () => {
+    const { driver, pageObjects } = await setupWebDriver();
+
+    await driver.wait(botConnected(), timeouts.directLine);
+
+    const input = await driver.findElement(By.css('input[type="text"]'));
+
+    await input.sendKeys('suggested-actions others', Key.RETURN);
+    await driver.wait(allOutgoingActivitiesSent(), timeouts.directLine);
+
+    await driver.wait(minNumActivitiesShown(3), timeouts.directLine);
+    await pageObjects.hideCursor();
+
+    const base64PNG = await driver.takeScreenshot();
+
+    expect(base64PNG).toMatchImageSnapshot(imageSnapshotOptions);
+  }, 60000);
 });

packages/core/src/sagas/incomingActivitySaga.js

@@ -26,13 +26,18 @@ function* observeActivity({ directLine, userID }) {
     yield put(incomingActivity(activity));
 
     // Update suggested actions
+    // TODO: [P3] We could put this logic inside reducer to minimize number of actions dispatched
     const messageActivities = yield select(activitiesOfType('message'));
     const lastMessageActivity = messageActivities[messageActivities.length - 1];
 
     if (activityFromBot(lastMessageActivity)) {
-      const { suggestedActions: { actions } = {} } = lastMessageActivity;
+      const { suggestedActions: { actions, to } = {} } = lastMessageActivity;
 
-      yield put(setSuggestedActions(actions));
+      if (!to || !to.length || to.includes(userID)) {
+        yield put(setSuggestedActions(actions));
+      } else {
+        yield put(setSuggestedActions());
+      }
     }
   });
 }