Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2463)

Issue 110089: Myspace specific addtions(statusmood & notifications) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 8 months ago by Harakiro
Modified:
8 years, 7 months ago
Reviewers:
chabotc, Arne Roomann-Kurrik, alexsisgonzalez41
Base URL:
http://opensocial-php-client.googlecode.com/svn/trunk/
Visibility:
Public.

Description

This patch adds two new myspace specific extensions to opensocial. Statusmood and Notifications. I also included the image file necessary for the mediaItem upload test.

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+354 lines, -4 lines) Patch
src/examples/images.jpg View 0 chunks +-1 lines, --1 lines 0 comments Download
src/examples/index.php View 1 chunk +2 lines, -0 lines 1 comment Download
src/examples/notifications.php View 1 chunk +52 lines, -0 lines 1 comment Download
src/examples/statusmood.php View 1 chunk +52 lines, -0 lines 0 comments Download
src/osapi/io/osapiRequest.php View 2 chunks +4 lines, -2 lines 0 comments Download
src/osapi/model/osapiModel.php View 1 chunk +3 lines, -2 lines 0 comments Download
src/osapi/model/osapiNotification.php View 1 chunk +40 lines, -0 lines 1 comment Download
src/osapi/model/osapiStatusMoodModel.php View 1 chunk +30 lines, -0 lines 0 comments Download
src/osapi/osapi.php View 1 chunk +3 lines, -1 line 0 comments Download
src/osapi/service/osapiNotifications.php View 1 chunk +83 lines, -0 lines 0 comments Download
src/osapi/service/osapiService.php View 1 chunk +2 lines, -0 lines 0 comments Download
src/osapi/service/osapiStatusMood.php View 1 chunk +84 lines, -0 lines 0 comments Download

Messages

Total messages: 3
Harakiro
14 years, 8 months ago (2009-08-26 21:18:47 UTC) #1
Arne Roomann-Kurrik
Aside from some minor formatting issues, this looks good. Commit whenever you're ready. http://codereview.appspot.com/110089/diff/1/3 File ...
14 years, 8 months ago (2009-08-26 22:46:10 UTC) #2
AlexsisGonzalez41
8 years, 7 months ago (2015-09-28 02:56:47 UTC) #3
Message was sent while issue was closed.

          
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b