Re: [asterisk-dev] [Code Review] 3240: res_pjsip_exten_state: Presence for digium phones

2014-02-27 Thread Kevin Harwell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3240/ --- (Updated Feb. 27, 2014, 3:19 p.m.) Status -- This change has been

Re: [asterisk-dev] [Code Review] 3240: res_pjsip_exten_state: Presence for digium phones

2014-02-26 Thread Joshua Colp
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3240/#review10961 --- Ship it! Ship It! - Joshua Colp On Feb. 21, 2014, 5 p.m.,

Re: [asterisk-dev] [Code Review] 3240: res_pjsip_exten_state: Presence for digium phones

2014-02-21 Thread Kevin Harwell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3240/ --- (Updated Feb. 21, 2014, 11 a.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 3240: res_pjsip_exten_state: Presence for digium phones

2014-02-21 Thread Kevin Harwell
On Feb. 20, 2014, 5:38 p.m., Mark Michelson wrote: asterisk/trunk/tests/channels/pjsip/subscribe/digium_presence/run-test, lines 39-40 https://reviewboard.asterisk.org/r/3240/diff/1/?file=54248#file54248line39 I recommend using AMI SetVar here instead. The big reason is that it

Re: [asterisk-dev] [Code Review] 3240: res_pjsip_exten_state: Presence for digium phones

2014-02-20 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3240/#review10915 ---

[asterisk-dev] [Code Review] 3240: res_pjsip_exten_state: Presence for digium phones

2014-02-19 Thread Kevin Harwell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3240/ --- Review request for Asterisk Developers. Repository: testsuite