Part of the migration of some of the OpenSocial components to activity streams involves namespacing some of the OpenSocial properties that do not match to Activity Stream properties. This patch adds the openSocial namespace to the activity streams implementation so we can begin to build out these OpenSocial extensions.
Hi Ryan, Could you change the base URL for the codereview from "http://svn.apache.org/repos/asf/incubator/shindig/trunk/" to http://svn.apache.org/repos/asf/shindig/trunk/ ...
14 years, 9 months ago
(2011-04-13 18:42:37 UTC)
#2
Hi Ryan,
Could you change the base URL for the codereview from
"http://svn.apache.org/repos/asf/incubator/shindig/trunk/" to
http://svn.apache.org/repos/asf/shindig/trunk/ ?
- Henry
On Wed, Apr 13, 2011 at 10:37 AM, <rbaxter85@gmail.com> wrote:
> Reviewers: woodser_us.ibm.com,
>
> Description:
> Part of the migration of some of the OpenSocial components to activity
> streams involves namespacing some of the OpenSocial properties that do
> not match to Activity Stream properties. This patch adds the openSocial
> namespace to the activity streams implementation so we can begin to
> build out these OpenSocial extensions.
>
> Please review this at http://codereview.appspot.com/4398045/
>
> Affected files:
> content/sampledata/canonicaldb.json
>
java/social-api/src/main/java/org/apache/shindig/social/core/model/ActivityEntryImpl.java
>
java/social-api/src/main/java/org/apache/shindig/social/core/model/OpenSocialImpl.java
>
java/social-api/src/main/java/org/apache/shindig/social/core/util/xstream/XStream081Configuration.java
>
java/social-api/src/main/java/org/apache/shindig/social/opensocial/model/ActivityEntry.java
>
java/social-api/src/main/java/org/apache/shindig/social/opensocial/model/OpenSocial.java
>
java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonGroup.json
>
java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonId.json
>
java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonIds.json
>
java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonUpdated.json
>
>
>
--
Thanks,
Henry
Ah thats why the diffs weren't working. Should be all set now. On 2011/04/13 18:42:37, ...
14 years, 9 months ago
(2011-04-13 18:48:12 UTC)
#3
Ah thats why the diffs weren't working. Should be all set now.
On 2011/04/13 18:42:37, henry.saputra wrote:
> Hi Ryan,
>
> Could you change the base URL for the codereview from
> "http://svn.apache.org/repos/asf/incubator/shindig/trunk/" to
> http://svn.apache.org/repos/asf/shindig/trunk/ ?
>
> - Henry
>
> On Wed, Apr 13, 2011 at 10:37 AM, <mailto:rbaxter85@gmail.com> wrote:
> > Reviewers: http://woodser_us.ibm.com,
> >
> > Description:
> > Part of the migration of some of the OpenSocial components to activity
> > streams involves namespacing some of the OpenSocial properties that do
> > not match to Activity Stream properties. This patch adds the openSocial
> > namespace to the activity streams implementation so we can begin to
> > build out these OpenSocial extensions.
> >
> > Please review this at http://codereview.appspot.com/4398045/
> >
> > Affected files:
> > content/sampledata/canonicaldb.json
> >
>
java/social-api/src/main/java/org/apache/shindig/social/core/model/ActivityEntryImpl.java
> >
>
java/social-api/src/main/java/org/apache/shindig/social/core/model/OpenSocialImpl.java
> >
>
java/social-api/src/main/java/org/apache/shindig/social/core/util/xstream/XStream081Configuration.java
> >
>
java/social-api/src/main/java/org/apache/shindig/social/opensocial/model/ActivityEntry.java
> >
>
java/social-api/src/main/java/org/apache/shindig/social/opensocial/model/OpenSocial.java
> >
>
java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonGroup.json
> >
>
java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonId.json
> >
>
java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonIds.json
> >
>
java/social-api/src/test/java/org/apache/shindig/social/dataservice/integration/fixtures/ActivityEntryJsonUpdated.json
> >
> >
> >
>
>
>
> --
> Thanks,
> Henry
Issue 4398045: Add the openSocial namespace to entries from the activity stream produced by Shindig
Created 14 years, 9 months ago by rbaxter85
Modified 11 years, 3 months ago
Reviewers: woodser_us.ibm.com, henry.saputra
Base URL: http://svn.apache.org/repos/asf/shindig/trunk/
Comments: 0