Add postNotification method to userOperations #58

Open
wants to merge 1 commit into from

4 participants

@dhaber

Hi,

The Facebook API now supports sending notifications from your app to a user. This commit adds a postNotifcation method to userOperations to allow this from within Spring Social. It includes the new interface method, the implementation and some tests.

You can find details from Facebook here: https://developers.facebook.com/docs/app_notifications/

Thanks!
Doug

@habuma
Spring member

Doug: Thanks for the contribution. I'll try to take a deeper look at this, review it, and merge it in the next couple of weeks.

The one thing that I notice is that it requires an App access token, thus FacebookTemplate would have to be constructed with the app access token instead of a user access token (which is not common or even ideal). I've got some thoughts on how to construct (or be given from the connect framework) a FacebookTemplate that has both a user access token for most operations and an app access token for those operations that need it. I've created https://jira.springsource.org/browse/SOCIALFB-101 to track this. I may attack SOCIALFB-101 and your pull request together, thus there may be some changes to your code to account for whatever I come up with for app access tokens.

Also, just to make sure: Have you signed the contribute agreement at https://jira.springsource.org/browse/SOCIALFB-101 yet? If not, the legal-minded people here would be happy if you did. Thanks.

@habuma
Spring member

Correction: THe link for the contributor agreement is at https://support.springsource.com/spring_committer_signup

@dhaber

Thanks @habuma ! I signed the agreement.

@mavarazy

What is the status of this?

@andrebrov

Any update since 2012?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment