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

WebChat tries to render all activity types #1358

Closed
v-kydela opened this issue Mar 7, 2019 · 6 comments · Fixed by #1363
Closed

WebChat tries to render all activity types #1358

v-kydela opened this issue Mar 7, 2019 · 6 comments · Fixed by #1363
Assignees
Labels
Bug Your classic code defect P0 Must Fix. Release-blocker

Comments

@v-kydela
Copy link

v-kydela commented Mar 7, 2019

Version

4.3.0

Describe the bug

The latest Emulator version and a few past versions have been trying to render endOfConversation activities and failing with a sort of error message

To Reproduce

The BotBuilder V3 Node SDK seems to generate endOfConversation activities when the bot encounters an error, though you can probably reproduce this in any SDK by sending your own activity

Expected behavior

The Emulator should not try to render anything but message activities in its conversation panel

Screenshots

image

Additional context

This is the same as #1328 but with endOfConversation activities instead of trace activities.

[bug]

@v-kydela v-kydela changed the title Emulator 4.3.0 tries to render endOfConversation activities Emulator 4.3.0 tries to render endOfConversation activities Mar 7, 2019
@a-b-r-o-w-n a-b-r-o-w-n changed the title Emulator 4.3.0 tries to render endOfConversation activities WebChat renders trace activities Mar 7, 2019
@a-b-r-o-w-n a-b-r-o-w-n added the Bug Your classic code defect label Mar 7, 2019
@a-b-r-o-w-n a-b-r-o-w-n changed the title WebChat renders trace activities WebChat renders all activities Mar 7, 2019
@a-b-r-o-w-n a-b-r-o-w-n changed the title WebChat renders all activities WebChat tries to render all activity types Mar 7, 2019
@pepperakos
Copy link

I can also confirm this same behavior... I have 4.3.0 of the bot emulator and had started an Enterprise Bot using the instructions from MS just yesterday.

@cwhitten cwhitten added 4.4 P0 Must Fix. Release-blocker labels Mar 8, 2019
@cwhitten
Copy link
Member

cwhitten commented Mar 8, 2019

To address this in the near term, the Emulator client will ignore trace and endOfConversation Activity types in the ActivityType middleware

@compulim
Copy link
Contributor

compulim commented Mar 8, 2019

Sample for activity middleware, https://github.com/Microsoft/BotFramework-WebChat/tree/master/samples/10.b.customization-password-input. Just return false for anything we don't want to render.

@corinagum
Copy link
Contributor

@justinwilaby started a Teams conversation where hiding the trace may not be an ideal solution. Reopening this issue for discussion and visibility.

@cwhitten
Copy link
Member

For visibility: The plan to address is that in Emulator mode, we don't render trace activity types, in sidecar mode we will, with implemented renderers for the trace, and specifically the debug trace type. What we won't do is render an activity whose type we do not have a renderer for all-up, of which the LUIS trace type, or endOfConversation is an example.

@cwhitten cwhitten assigned justinwilaby and unassigned a-b-r-o-w-n Mar 31, 2019
@cwhitten
Copy link
Member

cwhitten commented Apr 9, 2019

this is still addressed in #1363

@cwhitten cwhitten closed this as completed Apr 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Your classic code defect P0 Must Fix. Release-blocker
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants