Skip to content

Adding PinterestApi and PinterestExample#591

Merged
kullfar merged 3 commits intoscribejava:masterfrom
martinmalek:master
Dec 30, 2015
Merged

Adding PinterestApi and PinterestExample#591
kullfar merged 3 commits intoscribejava:masterfrom
martinmalek:master

Conversation

@martinmalek
Copy link
Copy Markdown
Contributor

Support for Pinterest Api

@martinmalek
Copy link
Copy Markdown
Contributor Author

Let me know if you want me to send you app id and secret for a quick test.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not so critical, but it's better not to use star imports.

@kullfar
Copy link
Copy Markdown
Member

kullfar commented Dec 28, 2015

Thanks for Pull request!
And yes, provide test credentials to email mentioned here please
https://github.com/scribejava/scribejava/blob/master/pom.xml#L48

@martinmalek
Copy link
Copy Markdown
Contributor Author

Comment changes applied. Please try again.

@kullfar
Copy link
Copy Markdown
Member

kullfar commented Dec 30, 2015

The code looks good. I'm trying to run the example and will merge after that.

kullfar added a commit that referenced this pull request Dec 30, 2015
Adding PinterestApi and PinterestExample
@kullfar kullfar merged commit 47bcf7b into scribejava:master Dec 30, 2015
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

Successfully merging this pull request may close these issues.

2 participants