Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Send typing indicator to Teams conversation #161

Merged

Conversation

AndrewPla
Copy link
Contributor

Description

I added the typing indicator per the documentation at https://docs.microsoft.com/en-us/azure/bot-service/rest-api/bot-framework-rest-connector-activities?view=azure-bot-service-4.0#typing

It is worth noting that I haven't been able to get it to work in a channel, just a DM. I didn't see anything in the documentation indicating that it's only supported in DM conversations, but I am unable to see typing indicators from other Users typing in channels. Feedback is appreciated on this part. The code will attempt to send the typing indicator whether or not the message was received from a DM.

Feedback is appreciated

Related Issue

Addresses Issue #127 - Feature Request: Send a typing indicator for Teams

Motivation and Context

This will allow you to receive a typing indicator in Teams. This is great for longer running commands because it let's the user know the message was received and the bot is working.

How Has This Been Tested?

I have been using this for a week or so in our environment. It sends a typing indicator to DM conversations but I'm not seeing a response when the bot replies to a channel.

Screenshots (if appropriate):

image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@devblackops
Copy link
Member

Awesome @AndrewPla. I'll check this out.

@devblackops
Copy link
Member

@AndrewPla This looks good to me! It's a shame Teams doesn't support this in normal channels.

I can't merge this while it's in draft mode. Can you change the status please?

From the looks of this user voice item, support in channels may be a ways off.
https://microsoftteams.uservoice.com/forums/555103-public/suggestions/18675718-display-user-is-typing-indication-also-in-teams

@AndrewPla AndrewPla marked this pull request as ready for review April 1, 2019 16:26
@AndrewPla
Copy link
Contributor Author

Draft status was removed.

@devblackops devblackops merged commit d50b033 into poshbotio:master Apr 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants