Add and Default RichContent for UI Rendering Purpose #195
Conversation
| public string? Payload { get; set; } | ||
| [JsonPropertyName("image_url")] | ||
| public string? ImageUrl { get; set; } | ||
| public string? PostBackUrl { get; set; } |
There was a problem hiding this comment.
Should we add [JsonPropertyName("postback_url")]?
| { | ||
| public class Recipient | ||
| { | ||
| public string? Id { get; set; } |
There was a problem hiding this comment.
The Id shouldn't be nullable.
| { | ||
| public class RichContent<T> | ||
| { | ||
| public Recipient? Recipient { get; set; } |
There was a problem hiding this comment.
Recipient should be required.
| /// RESPONSE | ||
| /// </summary> | ||
| [JsonPropertyName("messaging_type")] | ||
| public string? MessageingType { get; set; } |
There was a problem hiding this comment.
There is typo, and it's not nullable
There was a problem hiding this comment.
typo fixed; I observed that certain components, such as the button template, do not contain the messaging_type field. Refer to the example JSON below. Kindly inform me whether it is still necessary for our custom design."
There was a problem hiding this comment.
I think we can set it as a fixed value messaging_type="RESPONSE" currently.
|
|
||
| namespace BotSharp.Abstraction.Messaging.Models.RichContent | ||
| { | ||
| public class SenderActionModel |
There was a problem hiding this comment.
Can we rename it to SenderActionMessage?
| { | ||
| public string? Type { get; set; } | ||
| public string? Url { get; set; } | ||
| public string? Title { get; set; } |
There was a problem hiding this comment.
Type and Title should be required.
| { | ||
| public class CommonTemplate<T> | ||
| { | ||
| public string TemplateType { get; set; } = string.Empty; |
There was a problem hiding this comment.
[JsonPropertyName("template_type")]
| { | ||
| public class TextMessage | ||
| { | ||
| public string? Text { get; set; } |
|
|
No description provided.