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

[George Joseph]

George Joseph 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 5: Code-Review+1

(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>
> 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.

Looks like you removed the default?



-- 
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: 5
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, 29 Jun 2020 12:42:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: George Joseph <gjoseph at digium.com>
Comment-In-Reply-To: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200629/74b98d6d/attachment-0001.html>