[Asterisk-code-review] res pjsip/AMI: add contact.updated event (asterisk[13])

[George Joseph]

George Joseph has posted comments on this change.

Change subject: res_pjsip/AMI: add contact.updated event
......................................................................


Patch Set 1:

> Yes, for short period of time the sate is UPDATED.
 > live log:
 > [2016-04-08 16:49:39.696] VERBOSE[15596] res_pjsip/pjsip_configuration.c:
 > Contact test/sip:test at 192.168.10.1 is now Updated.  RTT: 0.000 msec
 > [2016-04-08 16:49:39.721] VERBOSE[15596] res_pjsip/pjsip_configuration.c:
 > Contact test/sip:test at 192.168.10.1 is now Unknown.  RTT: 0.000 msec
 > 
 > I don't know how to run test.

Here's the Test Suite Wiki:
https://wiki.asterisk.org/wiki/display/AST/Asterisk+Test+Suite+Documentation

It's worth it to set it up if you're going to be a frequent contributor.  For Asterisk 13, new features are supposed to have tests anyway and if something affects AMI especially, you'll want to use it to make sure you haven't broken anything.

Join #asterisk-dev on freenode irc and feel free to ask questions.

-- 
To view, visit https://gerrit.asterisk.org/2556
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8fad8aae9305481469c38d2146e1ba3a56d3108f
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Alexei Gradinari <alex2grad at gmail.com>
Gerrit-Reviewer: Alexei Gradinari <alex2grad at gmail.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <george.joseph at fairview5.com>
Gerrit-HasComments: No