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

Added Flickr sharer service #193

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

Conversation

Muppets
Copy link

@Muppets Muppets commented Mar 29, 2011

Modified Config.h to include Flickr API keys

@bluesuedesw
Copy link

Big ups to Muppets! Thanks so much for sharing. This works great. Couple of tips for future users:

1 - This requires the CFNetwork.framework
2 - One needs to setup the flickr app as a "web service" on the flickr authentication flow settings, and enter in your app's custom callback URL scheme.
3 - make sure you define and create the same URL scheme in your apps info.plist. It can be as simple as yourapp://flickr

@Eric-Wright
Copy link

Great work, Muppets! And thanks to bluesuedesw for the clarification, that helped me get this working quickly.

I noticed that LFSiteReachability.h/.m and NSData_LFHTTPFormExtensions.h/.m were unused, so I removed those files from my local copy.

I also modified SHKFlickr -sendPhoto: to not set the is_public flag. For my application, I want new photos to inherit the user's default sharing settings, rather than forcing all new photos to be private.

With feedback from Eric-Wright, Removed unused files
NSData_LFHTTPFormExtensions, LFWebAPIKit, LFSiteReachability. Also
removed is public flag to fall back to users default when upload
pictures.

This branch also seems to have issues building Evernote so removed
reference to this
@Muppets
Copy link
Author

Muppets commented Sep 23, 2011

Thanks Eric for your feedback. I've made your changes and pushed up a new commit.

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.

3 participants