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

Created XML tags using System.Xml #6312

Closed
wants to merge 11 commits into from
Closed

Conversation

breakingram
Copy link
Contributor

@breakingram breakingram commented Apr 28, 2022

Fixes #5792

Description

Ensure the appropriate SSML tags are set on the Activity Speak property using System.Xml.Linq classes
such as XDocument & XElement.

Testing

Voice is added to Speak property

AddVoice

Input:

Output:

image

Custom voice tag

AddVoice_NullSpeak

Voice is added with no speak tag

Input:

Output:

image

Multiple voice tags

AddMultiVoice

Input:

Output:

image

Custom speak is added with voice tag

AddSpeakWithVoice

Input:

Output:

image

Custom speak is added with no voice tag

AddSpeak_NullVoice

Input:

Output:

image

All SetSpeakMiddlewareTest

SetSpeakMiddlewareTests.cs

@ghost
Copy link

ghost commented Apr 28, 2022

CLA assistant check
All CLA requirements met.

@breakingram breakingram changed the title Create XML using XDocument Created XML tags using System.Xml Apr 29, 2022
@coveralls
Copy link
Collaborator

coveralls commented May 1, 2022

@breakingram breakingram marked this pull request as ready for review May 2, 2022 03:19
@breakingram breakingram requested a review from a team as a code owner May 2, 2022 03:19
@breakingram breakingram added the Automation: No parity PR does not need to be applied to other languages. label May 2, 2022
@EricDahlvang EricDahlvang added Automation: Parity with js The PR needs to be ported to JS and removed Automation: No parity PR does not need to be applied to other languages. labels Jun 13, 2022
@breakingram breakingram marked this pull request as draft June 30, 2022 16:08
@breakingram breakingram force-pushed the ramfattah/SetSpeakMiddleware branch from b52e08f to a7a4cf0 Compare June 30, 2022 18:21
@breakingram breakingram marked this pull request as ready for review July 2, 2022 05:15
@BruceHaley
Copy link
Contributor

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll

breakingram and others added 5 commits July 17, 2022 23:08
* Changes raw string to XDocument to create xml

* Parsed string xml in speak tag

* remove unnecessary imports

* Updated unit test & code refactor

* Multiple voice tags can be added in speak field

* function name update

* Check for valid XML tags to reduce processing

* Add unit tests for SetSpeakMiddleware
@BruceHaley
Copy link
Contributor

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Automation: Parity with js The PR needs to be ported to JS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SetSpeakMiddleware uses string formatting to create XML
5 participants