Skip to content
This repository has been archived by the owner on Apr 5, 2022. It is now read-only.

Added Linkedin Connect and fixed few compilation errors #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

shravanmahankali
Copy link

Hi There,

Added linkedin connect and replaced following no more available classes causing compilation failures:
-> "org.springframework.social.connect.signin.web.ProviderUserSignInController" class with "org.springframework.social.connect.signin.web.ProviderSignInController",
-> "org.springframework.social.connect.signin.web.ProviderUserSignInUtils" class with "org.springframework.social.connect.signin.web.ProviderSignInUtils".

NOTE: Please add the SpringSource "linkedin.consumerKey" and "linkedin.consumerSecret" in the following file src\main\webapp\WEB-INF\spring\properties.xml to make linkedin connect working.

PS: Here I have briefed my work - http://shravanmahankali.wordpress.com/2011/04/21/spring-social-and-linkedin/ (this draft is based on my work on spring-social 1.0.0.M2 version, which is not committed due to various differences.)

Let me know if need any information.

Thank You,
Shravan

shravanmahankali added 2 commits April 25, 2011 16:32
…es causing compilation failures:

"org.springframework.social.connect.signin.web.ProviderUserSignInController" class with "org.springframework.social.connect.signin.web.ProviderSignInController",
"org.springframework.social.connect.signin.web.ProviderUserSignInUtils" class with "org.springframework.social.connect.signin.web.ProviderSignInUtils"
@@ -54,7 +54,7 @@ public class SignupController {
}
boolean accountCreated = createAccount(form, formBinding);
if (accountCreated) {
ProviderUserSignInUtils.handlePostSignUp(request);
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ProviderUserSignInUtils is being replaced by ProviderSignInUtils

@shravanmahankali
Copy link
Author

would anyone review this change and merge to main source?

@kdonald
Copy link

kdonald commented May 2, 2011

Have you merged with the latest changes in master? Some of the changes I'm seeing have already been applied. Need to make sure your changes can be applied on top of the latest. See Github help & http://blog.springsource.com/2010/12/21/social-coding-in-spring-projects/ for some guidelines.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants