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

Add Flickr-specific sample code to the README #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

deg
Copy link

@deg deg commented Dec 4, 2016

This PR is in response to #28

I'm not completely happy with it but, realistically, I'm not going to have time to do much better anytime soon. Even as is, this does contain helpful info for anyone trying to work with Flickr. But, I'll completely understand if you want to move this into a separate file rather than leaving it in your main README.org.

Problems:

  • This is not fully tested. The code is cut-n-pasted from my working app, but I have not tested in isolation. I was careful, but may have missed some bits.
  • The flow is different than in your Twitter sample. This is from the server side of a Compojure-based client/server app.

That said, there is value here. I struggled for a while before I figured out how to get oauth-clj to work in my app and with Flickr. This captures what I learned and will, I hope, be useful for the next person who tries.

@deg deg mentioned this pull request Dec 4, 2016
@r0man
Copy link
Owner

r0man commented Dec 8, 2016

@deg What about adding this to the Wiki instead of the README?

@deg
Copy link
Author

deg commented Dec 8, 2016

That would be fine.

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