[Asterisk-code-review] manager - Add Content-Type parameter to the SendText action (asterisk[13])

[Kevin Harwell]

Kevin Harwell has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/14517 )

Change subject: manager - Add Content-Type parameter to the SendText action
......................................................................


Patch Set 3:

(1 comment)

https://gerrit.asterisk.org/c/asterisk/+/14517/3/main/manager.c 
File main/manager.c:

https://gerrit.asterisk.org/c/asterisk/+/14517/3/main/manager.c@881 
PS3, Line 881: 			<parameter name="Content-Type" required="false" default="text">
             : 				<para>The type of content in the message</para>
             : 			</parameter>
> Where is the default set and shouldn't it be "text/plain"?
I thought the core set it to "text" if the content type is not set, but looking again it uses "" if content type is not specified. So I'll set the default to that.



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/14517
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: Ibb5315702532c6b954e1498beddc8855fabdf4bb
Gerrit-Change-Number: 14517
Gerrit-PatchSet: 3
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Comment-Date: Mon, 22 Jun 2020 14:43:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200622/cf6b9e6a/attachment.html>