Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-09-20 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1689/#review1978 --- Ship it! Committed revision 1173325. - Ryan On 2011-09-16 20:3

Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-09-16 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1689/ --- (Updated 2011-09-16 20:36:03.558948) Review request for shindig, Ryan Baxter, Ma

Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-09-08 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1689/ --- (Updated 2011-09-08 14:01:47.339559) Review request for shindig, Ryan Baxter, Ma

Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-09-01 Thread Stanton Sievers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1689/#review1716 --- Ship it! LGTM - Stanton On 2011-08-31 22:12:15, Dan Dumont wrote:

Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-08-31 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1689/ --- (Updated 2011-08-31 22:12:15.636913) Review request for shindig, Ryan Baxter and

Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-08-31 Thread Dan Dumont
> On 2011-08-31 21:43:06, Ryan Baxter wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/actions/actions_container.js, > > line 789 > > > > > > Any particular reason why you ch

Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-08-31 Thread Dan Dumont
> On 2011-08-31 21:30:42, Stanton Sievers wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/actions/actions_container.js, > > line 785 > > > > > > Can you also verify typeof c

Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-08-31 Thread Dan Dumont
> On 2011-08-31 21:30:42, Stanton Sievers wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/actions/actions_container.js, > > line 785 > > > > > > Can you also verify typeof c

Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-08-31 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1689/#review1705 --- http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javas

Re: Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-08-31 Thread Stanton Sievers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1689/#review1706 --- http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javas

Review Request: Implement Specification: Issue 1210: Extend actions feature to allow container to register listeners for when actions are run

2011-08-31 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1689/ --- Review request for shindig, Ryan Baxter and Stanton Sievers. Summary --- Sp