search

eyhan / facebook-java-api

Automatically exported from code.google.com/p/facebook-java-api
0 stars 0 forks source link

Patch for new DashBoard API #288

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Against version: 3.0.3-SNAPSHOT

Just to get the ball rolling, I have supplied the patch for the following 
dashboard API methods:

dashboard.multiAddNews
dashboard.addGlobalNews
dashboard.publishActivity
dashboard.multiIncrementCount

I have documented the places where a FB bug was encountered and the 
corresponding FB entry. Further changes will be needed on the response 
processing side of things once the bug report is fixed. I am planning on 
adding some additional API calls in addition to fixing any issues that 
surface relating to the above calls. Could someone integrate this to trunk 
or give me commit priviledges? 

At a future point in time (once the FB fire drill is done), I am planning 
on submitting an implementation for the Banned User API.

Thanks!

-Kyle

Original issue reported on code.google.com by klom...@gmail.com on 1 Feb 2010 at 10:38

Attachments:

GoogleCodeExporter commented 8 years ago 
There is also a contribution here with no overlap (other than the Data objects 
used):

http://code.google.com/p/facebook-java-api/issues/detail?id=280

Original comment by klom...@gmail.com on 2 Feb 2010 at 12:49

GoogleCodeExporter commented 8 years ago 
Added the dashboard.clearGlobalNews call. I have attached the full patch 
including 
this new change.

Original comment by klom...@gmail.com on 2 Feb 2010 at 2:45

Attachments:

GoogleCodeExporter commented 8 years ago 
I will eventually have time to apply this patch, but Kyle sits 5 feet from me 
and
does great work, so it would be super if someone could cut out the middleman 
and give
Kyle commit privileges.  Thanks.

Original comment by jasper.r...@gmail.com on 2 Feb 2010 at 6:58

GoogleCodeExporter commented 8 years ago 
FB changed the format of the parameter for the dashboard.publishActivity 
method. I 
have included the latest cumulative patch with those changes applied.

Original comment by klom...@gmail.com on 4 Feb 2010 at 4:17

Attachments:

GoogleCodeExporter commented 8 years ago 
Hey, someone will commit these patches to trunk? Or what's going on?

Original comment by rdmytre...@gmail.com on 5 Feb 2010 at 8:13

GoogleCodeExporter commented 8 years ago 
I have committed this patch.

Original comment by jasper.r...@gmail.com on 5 Feb 2010 at 3:48

GoogleCodeExporter commented 8 years ago 
Issue 280 has been merged into this issue.

Original comment by jasper.r...@gmail.com on 5 Feb 2010 at 3:48

GoogleCodeExporter commented 8 years ago 
Thanks, Jasper.

Original comment by klom...@gmail.com on 5 Feb 2010 at 3:51

GoogleCodeExporter commented 8 years ago 
Any chance for a new maven SNAPSHOT with this revision soon?

Original comment by ori.da...@gmail.com on 7 Feb 2010 at 1:06