BotFramework-WebChat icon indicating copy to clipboard operation
BotFramework-WebChat copied to clipboard

Anjagraw/typing indicator

Open anjalinitd opened this issue 3 years ago • 2 comments

Fixes #

Changelog Entry

Added a tag attribute for from object for typing indicator.

Description

  • BotFramework-webchat-adapter sends from object as part of typing indicator activity which is mapped to typing object in Botframeork-webchat. Added a new tag in adapter as well as webchat to pass tag to receiver side to indicate sender is typing in public/Internal panel. (We have a single adapter for two react webchat's one is public other is internal to distinguish between the two we need tag params)

Design

Specific Changes

  • Added a tag attribute for typing object in api/hooks
  • Mapped to tag for from object for any incoming_activity for typing indicator

-

  • [x] I have added tests and executed them locally
  • [x] I have updated CHANGELOG.md
  • [x] I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • [x] Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • [x] Browser and platform compatibilities reviewed
  • [x] CSS styles reviewed (minimal rules, no z-index)
  • [x] Documents reviewed (docs, samples, live demo)
  • [x] Internationalization reviewed (strings, unit formatting)
  • [x] package.json and package-lock.json reviewed
  • [x] Security reviewed (no data URIs, check for nonce leak)
  • [x] Tests reviewed (coverage, legitimacy)

anjalinitd avatar Mar 30 '22 10:03 anjalinitd

Direct Line activity schema does not contain from.tag field, https://github.com/microsoft/botframework-sdk/blob/master/specs/botframework-activity/botframework-activity.md#from. So, we can't retrieve tag value from the from property.

We can consider retrieving/saving channelData to the structure returned by useActiveTyping.

Please fill in the PR form. You can look for other PRs to get an idea of details we want to have. This is critical for us. If a future regression happens (it do!), the details in the PR form will help us to understand how this feature should work and why we added it in first place.

compulim avatar Apr 05 '22 22:04 compulim

Direct Line activity schema does not contain from.tag field, https://github.com/microsoft/botframework-sdk/blob/master/specs/botframework-activity/botframework-activity.md#from. So, we can't retrieve tag value from the from property. -> From object comes from botframework-webchat-adapter as per my understanding and then we map in webchat. We have a PR to update tag for from object in adapter side as well. I have tested this and it's working fine. Can updated the read me file: https://github.com/microsoft/botframework-webchat-chat-adapter/pull/49

We can consider retrieving/saving channelData to the structure returned by useActiveTyping.

Please fill in the PR form. You can look for other PRs to get an idea of details we want to have. This is critical for us. If a future regression happens (it do!), the details in the PR form will help us to understand how this feature should work and why we added it in first place.

anjalinitd avatar Apr 06 '22 06:04 anjalinitd