> On 2012-04-12 12:09:52, Ryan Baxter wrote: > > LGTM, have you signed up for Apache/s JIRA deployment? I want to assign > > the JIRA to you and have you upload the final patch to the JIRA granting > > the ASF license before I submit the code. I tried to look for you to > > assign the JIRA to you but I didn't see you in the JIRA system. If you > > have not registered could you? > > > > https://issues.apache.org/jira/browse/SHINDIG
Hi Ryan, My username is yaozhang. I have upload the patch to https://issues.apache.org/jira/browse/SHINDIG-1698 and grant the ASF license. - Yao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4680/#review6873 ----------------------------------------------------------- On 2012-04-12 01:47:45, Yao Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4680/ > ----------------------------------------------------------- > > (Updated 2012-04-12 01:47:45) > > > Review request for shindig, Ryan Baxter, Eric Woods, and Stanton Sievers. > > > Summary > ------- > > This is the patch for https://reviews.apache.org/r/3764 as xuli is not > working on it. > @Ryan, I have merged line 58 and 59 of > /trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/spi/CollectionOptions.java > and break line 60 into multiple lines. As predefinedParameters is using > RequestItem, it can not be changed to private static final. > > > This addresses bug SHINDIG-1698. > https://issues.apache.org/jira/browse/SHINDIG-1698 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/spi/CollectionOptionsTest.java > PRE-CREATION > > http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/BaseRequestItem.java > 1311182 > > http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/RequestItem.java > 1311182 > > http://svn.apache.org/repos/asf/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/spi/CollectionOptions.java > 1311182 > > Diff: https://reviews.apache.org/r/4680/diff > > > Testing > ------- > > done > > > Thanks, > > Yao > >
