Re: Review Request: Actions, selection, and open-views feature issues in a locked domain environment.

2011-12-06 Thread Matthew Hatem
gt; > > Review request for shindig, Ryan Baxter, Jason Chiang, Matthew Hatem, and > Stanton Sievers. > > > Summary > --- > > As a general rule, any time anyone is writing a feature using RPC, those > endpoints should never expect to get a function (or someth

Re: Review Request: Add additional lifecycle handlers

2011-11-21 Thread Matthew Hatem
; > (Updated 2011-11-18 21:56:17) > > > Review request for shindig, Matthew Hatem and Igor Belakovskiy. > > > Summary > --- > > I have added additional lifecycle handlers. My primary use case here is to > be able to time the different events that take pla

Re: Review Request: Containers should be able to override a subset of the rpc handlers of the actions feature.

2011-10-18 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2424/ --- (Updated 2011-10-18 19:16:00.617784) Review request for shindig, Ryan Baxter and

Review Request: Containers should be able to override a subset of the rpc handlers of the actions feature.

2011-10-18 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2424/ --- Review request for shindig, Ryan Baxter and Jason Chiang. Summary --- Conta

Re: Review Request: Selection feature implementation does not match spec

2011-08-23 Thread Matthew Hatem
> How does this ever get called? Shouldn't this be called when the > > gadget calles removeListener? If so it should be called from your router > > function...right? > > Matthew Hatem wrote: > This allows the container to add and remove listeners, h

Re: Review Request: Selection feature implementation does not match spec

2011-08-23 Thread Matthew Hatem
> Shouldn't there be a test for removeListener as well? Yes I will add one. - Matthew --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1615/#review1604 -

Review Request: Selection feature implementation does not match spec

2011-08-23 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1615/ --- Review request for shindig and Ryan Baxter. Summary --- https://issues.apac

Re: Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-20 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1533/ --- (Updated 2011-08-21 03:17:07.524460) Review request for shindig and Ryan Baxter.

Re: Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-20 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1533/ --- (Updated 2011-08-20 17:31:02.792917) Review request for shindig and Ryan Baxter.

Re: Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-19 Thread Matthew Hatem
t; Same here the function is called registerHidActionsListener > > Matthew Hatem wrote: > The Core Gadget spec is wrong and inconsistent. The API should be > symmetric with the CommonContainer API which uses 'Handler' not 'Listener'. > >

Re: Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-19 Thread Matthew Hatem
eed to change the name of these as well Done - Matthew --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1533/#review1547 ------- On 2

Re: Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-19 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1533/ --- (Updated 2011-08-19 13:56:41.695019) Review request for shindig and Ryan Baxter.

Re: Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-18 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1533/ --- (Updated 2011-08-19 01:53:08.369683) Review request for shindig and Ryan Baxter.

Re: Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-18 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1533/ --- (Updated 2011-08-19 00:33:54.116967) Review request for shindig and Ryan Baxter.

Re: Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-18 Thread Matthew Hatem
che.org/r/1533/diff/2/?file=32941#file32941line584> > > > > Same here put takes 2 parameters should be push(), sorry - Matthew --- This is an automatically generated e-mail. To reply, visit: https://rev

Re: Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-16 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1533/ --- (Updated 2011-08-16 19:17:05.813686) Review request for shindig and Ryan Baxter.

Review Request: Fixes to actions feature to match spec (runAction and registerXHandler).

2011-08-16 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1533/ --- Review request for shindig and Ryan Baxter. Summary --- Fixes to actions fe

Re: Review Request: Fix to support for VIEW and VIEW_TARGET in actions feature

2011-08-15 Thread Matthew Hatem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1201/ --- (Updated 2011-08-15 15:08:16.073843) Review request for shindig, Ryan Baxter and

Question about access to cached gadget metadata

2011-07-27 Thread Matthew Hatem
We have a use case where I have my own instance of the Common Container and I need access to a gadgets' metadata outside the context of gadget navigation. Currently the common container keeps a cache/map of gadget metadata. We see two potential solutions: 1) Create a lifecycle callback and cache