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

Adds Pocket Support #2

Merged
merged 1 commit into from
Jun 30, 2013
Merged

Adds Pocket Support #2

merged 1 commit into from
Jun 30, 2013

Conversation

ZachOrr
Copy link
Contributor

@ZachOrr ZachOrr commented Jun 29, 2013

Adds "Save to Pocket" as an option in the UIActivityViewController

PocketAPIActivity taken from here Pocket/Pocket-ObjC-SDK#1

To get the functionality to work properly, you'll need to create two Pocket applications (one for the iPhone, one for the iPad) http://getpocket.com/developer/apps/

Modify the Constants.h file with the keys for each device. Additionally, inside the Info.plist file, you'll need to edit URL Types -> Item 0 -> URL Schemes -> Item 0 to be pocketappXXXX, where XXXX is the digits before the dash in your consumer keys

Ex:

Consumer Key: 42-abcdef
pocketapp42

The one thing this pull request is missing is some indicator about successfully saving a link or a link failing to being saved. Right now there are just NSLogs. I figured a stock UIAlertView wouldn't fit the style of the app and would be too intrusive. Any suggestions?

@FredKSchott
Copy link

👍

mmackh added a commit that referenced this pull request Jun 30, 2013
Merge proposed Pocket Support
@mmackh mmackh merged commit 3672a99 into mmackh:master Jun 30, 2013
@mmackh
Copy link
Owner

mmackh commented Jun 30, 2013

Thank you very much for this contribution - as for the successful saving to pocket, I don't think it needs any kind of notification, since the SDK redirects the user to their app. Will continue to think about something though.

Edit: Just saw that it doesn't redirect after I've authenticated the app

@ZachOrr
Copy link
Contributor Author

ZachOrr commented Jun 30, 2013

Just tested it - it redirects fine for me

@mmackh
Copy link
Owner

mmackh commented Jun 30, 2013

My point was that it doesn't always redirect, but still saves the article i.e. giving the user no indication that the article was successfully saved.

@ZachOrr
Copy link
Contributor Author

ZachOrr commented Jun 30, 2013

Aaah gotcha

-- 
Zachary Orr

On June 30, 2013 at 11:14:11 AM, Maximilian Mackh (notifications@github.com) wrote:

My point was that it doesn't always redirect, but still saves the article i.e. giving the user no indication that the article was successfully saved.


Reply to this email directly or view it on GitHub.

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