[
https://issues.apache.org/jira/browse/SHINDIG-1698?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13253330#comment-13253330
]
[email protected] commented on SHINDIG-1698:
--------------------------------------------------------
bq. On 2012-04-12 12:09:52, Ryan Baxter wrote:
bq. > 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?
bq. >
bq. > https://issues.apache.org/jira/browse/SHINDIG
bq.
bq. Yao Zhang wrote:
bq. Hi Ryan,
bq.
bq. My username is yaozhang. I have upload the patch to
https://issues.apache.org/jira/browse/SHINDIG-1698 and grant the ASF license.
Committed please close the review. Thanks.
- Ryan
-----------------------------------------------------------
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:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/4680/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-04-12 01:47:45)
bq.
bq.
bq. Review request for shindig, Ryan Baxter, Eric Woods, and Stanton Sievers.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. This is the patch for https://reviews.apache.org/r/3764 as xuli is not
working on it.
bq. @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.
bq.
bq.
bq. This addresses bug SHINDIG-1698.
bq. https://issues.apache.org/jira/browse/SHINDIG-1698
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/spi/CollectionOptionsTest.java
PRE-CREATION
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/BaseRequestItem.java
1311182
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/RequestItem.java
1311182
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/spi/CollectionOptions.java
1311182
bq.
bq. Diff: https://reviews.apache.org/r/4680/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. done
bq.
bq.
bq. Thanks,
bq.
bq. Yao
bq.
bq.
> Need to support query parameter for Social REST API
> ---------------------------------------------------
>
> Key: SHINDIG-1698
> URL: https://issues.apache.org/jira/browse/SHINDIG-1698
> Project: Shindig
> Issue Type: Bug
> Components: Java
> Affects Versions: 2.5.0-beta1
> Reporter: li xu
> Fix For: 2.5.0
>
> Attachments: parameter_patch.txt, parameter_patch_final.txt
>
>
> We have a use case that need to support optional parameters on Social REST
> API.
> Per open social spec discussion, provider specific parameter can be
> supported. Please see the link below for details.
> https://groups.google.com/forum/#!topic/opensocial-and-gadgets-spec/rOBmu4QGtwg
> This patch implemented optional parameter as a field in CollectionOptions.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
