Botbuilder-samples: Extremely small typo in code for inline attachment in HandlingAttachmentsBot

Created on 24 Oct 2018  路  2Comments  路  Source: microsoft/BotBuilder-Samples

Describe the bug

Missing @ before Name string in method GetInlineAttachment.
Causes the '\a' in "Resources\architecture-resize.png" to turn into an alert.

To Reproduce

Steps to reproduce the behavior:

  1. Run the sample project HandlingAttachmentsBot.
  2. Use debugger or Bot Framework Emulator when communicating with bot.
  3. Select '1. Inline Attachment' and see the \a turn into \u0007 by inspecting either the debugger value for Name, or the name property for the image in the emulator Inspector.

Expected behavior

Expect string Name to be "Resources\architecture-resize.png", and not "Resources\u0007rchitecture-resize.png".

Screenshots

From the source in AttachmentsBot.cs:
image

From the Inspector view in the Bot Framework Emulator:
image

Additional context

This is, of course, an extremely small problem, but hopefully it's also extremely easy to fix :)

[bug]

bug

Most helpful comment

Sure, I'll just have to find out how to do it first :)
Never done it before.
I'll look into it.

All 2 comments

Would you like to submit a pull request for this?

Sure, I'll just have to find out how to do it first :)
Never done it before.
I'll look into it.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

arunprasathv picture arunprasathv  路  3Comments

daveRendon picture daveRendon  路  6Comments

JoshBello picture JoshBello  路  6Comments

pesarkhobeee picture pesarkhobeee  路  9Comments

Batta32 picture Batta32  路  9Comments