Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do you plan on adding support for Plus Domains? #60

Open
nodoze opened this issue Sep 5, 2014 · 5 comments
Open

Do you plan on adding support for Plus Domains? #60

nodoze opened this issue Sep 5, 2014 · 5 comments

Comments

@nodoze
Copy link

nodoze commented Sep 5, 2014

Thanks for the good work. Very useful. I would be interested in assisting and adding Plus Domains capabilities such posting and and other circle management tasks. Thoughts?

@GabiAxel
Copy link
Collaborator

GabiAxel commented Sep 6, 2014

I welcome contributions. Fork the project, add support for G+ domains and submit a pull request. Please follow Spring's code style guidelines and write Javadoc and unit tests like the rest of the project. I suggest you create a "domains" package under "plus" and add your new content to it.

@nodoze
Copy link
Author

nodoze commented Sep 6, 2014

Thanks. I will give it a spin as time permits. Have you thought about integrating the google library to use their binding?

Sent from my iPad

On Sep 6, 2014, at 3:46 AM, Gabriel Axel notifications@github.com wrote:

I welcome contributions. Fork the project, add support for G+ domains and submit a pull request. Please follow Spring's code style guidelines and write Javadoc and unit tests like the rest of the project. I suggest you create a "domains" package under "plus" and add your new content to it.


Reply to this email directly or view it on GitHub.

@GabiAxel
Copy link
Collaborator

GabiAxel commented Sep 6, 2014

That's what Google.getAccessToken() is for - you can use Spring Social Google for the authentication process and use the access token with Google's Java library or any other code that interacts with the HTTP API.

Spring Social projects (not only this one) interact directly with the HTTP APIs, and the common core allows the specific projects to get a lot of free stuff out of the box, so there isn't much sense in wrapping another library to interact with the APIs. They can co-exist in the same project just fine as mentioned above.

@nodoze
Copy link
Author

nodoze commented Sep 6, 2014

I did some tests using the getAccessToken() approach yesterday. There are
some scope conflicts between the two APIs. Also it¹s not as clean as I¹d
like it to be since it creates competing bindings / model classes for the
same API. I¹m very familiar with most aspects of the spring framework and
very much appreciate the work the community is putting in to make free
software available to everyone. Since I¹ve been largely on the receiving
end till now I¹ve figured I would start paying back a bit. Anyway, I will
play around with and let you know how it goes.

From: Gabriel Axel notifications@github.com
Reply-To: GabiAxel/spring-social-google
<reply@reply.git
hub.com>
Date: Saturday, September 6, 2014 at 12:03 PM
To: GabiAxel/spring-social-google spring-social-google@noreply.github.com
Cc: Oliver Eberle oeberle@gmail.com
Subject: Re: [spring-social-google] Do you plan on adding support for Plus
Domains? (#60)

That's what Google.getAccessToken() is for - you can use Spring Social
Google for the authentication process and use the access token with Google's
Java library or any other code that interacts with the HTTP API.

Spring Social projects (not only this one) interact directly with the HTTP
APIs, and the common core allows the specific projects to get a lot of free
stuff out of the box, so there isn't much sense in wrapping another library
to interact with the APIs. They can co-exist in the same project just fine
as mentioned above.


Reply to this email directly or view it on GitHub
<#60 (comment)
24765> .

@vauvenal5
Copy link

I know this is not completely on topic but since you discussed here wrapping the Google API library I decided to continue here. The Spring Social documentation itself suggests integrating API bindings if they exist instead of writing new bindings: http://docs.spring.io/spring-social/docs/1.1.4.RELEASE/reference/htmlsingle/#integrating-an-existing-java-api-binding

So basically my question is, where is the advantage in providing another binding?

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

No branches or pull requests

3 participants