----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3053/#review3847 -----------------------------------------------------------
LGTM - Jason On 2011-12-07 16:53:27, Dan Dumont wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3053/ > ----------------------------------------------------------- > > (Updated 2011-12-07 16:53:27) > > > Review request for shindig, Ryan Baxter, Jason Chiang, and Stanton Sievers. > > > Summary > ------- > > Also updated the /samplecontainer/examples/embeddedexperiences/ container and > gadgets to demonstrate how one might support the feature. > > A spec update will be written up to account for these changes. > > > This addresses bug SHINDIG-1671. > https://issues.apache.org/jira/browse/SHINDIG-1671 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/embeddedexperiences/EEContainer.js > 1211472 > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/embeddedexperiences/PhotoList.xml > 1211472 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/open-views/viewenhancements-container.js > 1211478 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/open-views/viewEnhancements-test.js > 1211472 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/open-views/viewenhancements.js > 1211472 > > Diff: https://reviews.apache.org/r/3053/diff > > > Testing > ------- > > all tests pass, added a jsunit test case. > > > Thanks, > > Dan > >