[Asterisk-code-review] app_followme: Allow disabling FollowMe prompt (asterisk[13])

Dennis asteriskteam at digium.com
Wed Jul 29 04:35:35 CDT 2020


Dennis has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/14696 )

Change subject: app_followme: Allow disabling FollowMe prompt
......................................................................


Patch Set 2:

> Patch Set 1: Code-Review-1
> 
> (3 comments)
> 
> For new features, and improvements that are added to release branches we require some sort of test (Asterisk unit and/or testsuite test). Can this be tested via the Asterisk testsuite? Does it make sense to? I do see at least one test using app_followme in the Testsuite (tests/predial/).

I'm not familiar with test Asterisk testsuite but I'll try.


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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Change-Id: Ic15a2bac4f16d0fce7a1b5e7b375f9bafee37aa4
Gerrit-Change-Number: 14696
Gerrit-PatchSet: 2
Gerrit-Owner: Dennis <dennis.buteyn at xorcom.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Comment-Date: Wed, 29 Jul 2020 09:35:35 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200729/cbf8a93d/attachment.html>


More information about the asterisk-code-review mailing list