Skip to content

Fixed Adaptive Cards not speaking#2320

Merged
corinagum merged 3 commits intomicrosoft:masterfrom
tdurnford:2243
Aug 15, 2019
Merged

Fixed Adaptive Cards not speaking#2320
corinagum merged 3 commits intomicrosoft:masterfrom
tdurnford:2243

Conversation

@tdurnford
Copy link
Copy Markdown
Contributor

@tdurnford tdurnford commented Aug 15, 2019

Fixes #2243

Changelog Entry

Fix #2243. Fixed sagas to correctly mark activities with speaking attachments, by @tdurnford in PR #2320

Description

The speakActivityAndStartDictateOnIncomingActivityFromOthersSaga saga was not correctly marking activities with speaking attachments as speaking activities. Updating the if statement to check incoming activities' attachments for the speak property resolves the issue.


  • Testing Added

@coveralls
Copy link
Copy Markdown

coveralls commented Aug 15, 2019

Coverage Status

Coverage decreased (-0.08%) to 66.114% when pulling b8f8eab on tdurnford:2243 into 892e56e on microsoft:master.

@tdurnford tdurnford marked this pull request as ready for review August 15, 2019 18:46
Copy link
Copy Markdown
Contributor

@corinagum corinagum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@corinagum corinagum merged commit eaa1d22 into microsoft:master Aug 15, 2019
@compulim compulim mentioned this pull request Oct 4, 2019
13 tasks
@compulim compulim mentioned this pull request Oct 25, 2019
55 tasks
@compulim compulim mentioned this pull request Dec 10, 2019
73 tasks
@corinagum corinagum mentioned this pull request Dec 13, 2019
73 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adaptive Card speak problem

3 participants