----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4239/#review5735 -----------------------------------------------------------
The shindig-container was the old container code before existance of common container. We might want to remove it in favor of common container. The features shouldnt be used together. - Henry On 2012-03-08 06:31:13, hu jian wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4239/ > ----------------------------------------------------------- > > (Updated 2012-03-08 06:31:13) > > > Review request for shindig, Henry Saputra, li xu, and Dan Dumont. > > > Summary > ------- > > if shindig-container feature is included, it overwrites all of the > osapi.container functions created by container feature. > shindig.container/osapi.js should at the very least check that > osapi.container doesn't ready exist before overwriting it to {}. > shindig.container/feature.xml should also be altered to reflect the addition > of osapi.container.listMethods > > > This addresses bug SHINDIG-1724. > https://issues.apache.org/jira/browse/SHINDIG-1724 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/shindig.container/feature.xml > 1298262 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/shindig.container/osapi.js > 1298262 > > Diff: https://reviews.apache.org/r/4239/diff > > > Testing > ------- > > Done > > > Thanks, > > hu > >
