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

new pocket command #63

Closed
wants to merge 8 commits into from
Closed

new pocket command #63

wants to merge 8 commits into from

Conversation

s-frostick
Copy link
Contributor

@s-frostick s-frostick commented Mar 8, 2018

#29 Added pocket integration using https://github.com/motemen/go-pocket.

User needs to set a consumer key or create a new one; https://getpocket.com/developer/apps/new

Then can run the following commands

shiori pocket key <APIKEY>
to set the api key

shiori pocket login

The login command will open a browser where the user can login to pocket to authorize the request.

The accesstoken and consumer key are stored in $HOME/.shiori/config.json

Then can run
shiori pocket import
to actually import items from pocket

by default this will not import items that have been archived in pocket, for this run with

shiori pocket import --all

If just want to import items with a particular tag can use

shiori pocket import --tag <TAG>

Note: the import command imports tags with spaces from pocket although the current web interface is unable to handle these correctly.

@RadhiFadlillah
Copy link
Collaborator

Hi @s-frostick, thanks for the PR. However, I found this is too complicated for the user, because they need to create Pocket application first.

To solve this, I've made it so user can just import his Pocket's account by using the exported HTML file in a74773b. The only downside is it won't import the bookmark's title from Pocket. However, since (AFAIK) all title in Pocket is resolved from the webpage's metadata, user can easily do it himself by using shiori update command.

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.

None yet

2 participants