[Asterisk-code-review] documentation: Add missing AMI documentation (asterisk[master])

N A asteriskteam at digium.com
Mon Dec 13 08:43:01 CST 2021


N A has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/17650 )

Change subject: documentation: Add missing AMI documentation
......................................................................


Patch Set 1:

(1 comment)

Patchset:

PS1: 
One thing that seems a bit funny to me: QueueMemberStatus vs. QueueMember.

There is an AMI event of "QueueMember", but what it's sending seems exactly like what the QueueMemberStatus documentation describes. Was QueueMember changed to QueueMemberStatus at one point? In that case, I think the AMI event itself needs to be updated to reflect the documentation. Haven't touched that yet.

I also adding missing <managerEvent>s, at that seemed to be precluding some events from showing up in "manager show events", but that did not seem to do it. So not sure how to fix that, since it doesn't seem like events get registered, only actions.



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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I92b06b88c8cadc0155f95ebe3e870b3e795a8c64
Gerrit-Change-Number: 17650
Gerrit-PatchSet: 1
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-CC: Friendly Automation
Gerrit-Comment-Date: Mon, 13 Dec 2021 14:43:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20211213/6befb8d9/attachment.html>


More information about the asterisk-code-review mailing list