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

Composer + Telephony Package encode locale string with "\", making them non-funtional #9573

Closed
1 of 3 tasks
jameslew opened this issue Apr 27, 2023 · 2 comments
Closed
1 of 3 tasks
Assignees
Labels
customer-reported Required for internal Azure reporting. Do not delete. Needs-triage A new issue that require triage Type: Bug Something isn't working

Comments

@jameslew
Copy link

Describe the bug

When you put SSML tags inside of the speech variant node, occasionally a locale string needs to be included - say en-US. The Composer unfortunately is encoding those strings with a "", making them non-functional:

Please press 1 for English or 2 for Spanish. Stop Silence

We need to not encode these tags in the edit box, or decode them as well so they stay functional.

Version

Version: 2.1.2
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0

OS

  • macOS
  • Windows
  • Ubuntu

To Reproduce

Steps to reproduce the behavior:

Add the telephony package
Add a multi-choice with silence detection node
Add a speech variant
Paste the following into the box:

Please press 1 for English or 2 for Spanish. Stop Silence

Nave away and back to the node; it shows encoded locales.

Expected behavior

No Encoding applied

Additional context

Impacts users of Dynamics Omnichannel trying to build voice skills for DTMF handling.

@jameslew jameslew added Type: Bug Something isn't working Needs-triage A new issue that require triage customer-reported Required for internal Azure reporting. Do not delete. labels Apr 27, 2023
@JhontSouth
Copy link
Collaborator

We have been able to replicate the issue, but only when using multi-line SSML tags. This issue was reported here, but the PR with the fix is not yet merged.

@ceciliaavila ceciliaavila self-assigned this May 9, 2023
@tracyboehrer
Copy link
Member

Duplicate

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Required for internal Azure reporting. Do not delete. Needs-triage A new issue that require triage Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants