SDK Release 5.3.0 #482
SDK Release 5.3.0 #482
Conversation
…MessageContextGlobalSystem
…d removal of unnecessary model
… in TestListClassificationModel
| @@ -1825,7 +1828,7 @@ public DetailedResponse<AnalyzedDocument> AnalyzeDocument(string projectId, stri | |||
|
|
|||
| if (file != null) | |||
| { | |||
| var fileContent = new StreamContent(file); | |||
| var fileContent = new ByteArrayContent(file.ToArray()); | |||
There was a problem hiding this comment.
@kevinkowa when we reviewed this one together. I remember that we ignored this change.
I was about to revert back to var fileContent = new StreamContent(file); but also found that every other places use ByteArrayContent. So I'm wondering if we should just add this change since I did not see any test failure on DiscoveryService. But, I also don't know if it need to be kept as SteamContent. Please let me know if we want to convert this part back.
There was a problem hiding this comment.
Yes, lets discard this change. If this is no longer an issue we can check it out when we do a major release
There was a problem hiding this comment.
PR looks good after this change
There was a problem hiding this comment.
Okay! I updated the change. Thank you
| @@ -1825,7 +1828,7 @@ public DetailedResponse<AnalyzedDocument> AnalyzeDocument(string projectId, stri | |||
|
|
|||
| if (file != null) | |||
| { | |||
| var fileContent = new StreamContent(file); | |||
| var fileContent = new ByteArrayContent(file.ToArray()); | |||
There was a problem hiding this comment.
Yes, lets discard this change. If this is no longer an issue we can check it out when we do a major release
…ment to the original code
…tion to authenticated
|
@kevinkowa the PR is ready to be reviewed again. Thanks |
| /// Routing or other contextual information to be used by target service desk systems. | ||
| /// </summary> | ||
| [JsonProperty("transfer_info", NullValueHandling = NullValueHandling.Ignore)] | ||
| public DialogNodeOutputConnectToAgentTransferInfo TransferInfo { get; protected set; } |
There was a problem hiding this comment.
we can discard this change, TransferInfo is not required here
…neric for assistant-v2
|
🎉 This PR is included in version 5.3.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Service Changes
Assistant V1
alt_textproperty added to Image response type
Assistant V2
session_start_timeandstateproperties added toMessageContextGlobalSystemmodelalt_textproperty added to Image response typelocation property of RuntimeEntity model changed from required to optional
Discovery V2
Enumupdate forCreateProjectConstants
NLU
Fix for
listClassificationsModelsthrough return type change and removal of unnecessary model
STT
More languages supported for next generation models
TTS
New voice models added
CC, Disco V1, LT, NLC, PI, TA, VisRec V3 & V4
No changes