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 an API for search/add/edit/delete link #115

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

Conversation

leblanc-simon
Copy link

This pull request add an REST API to Shaarli

Documentation and tests : https://gist.github.com/leblanc-simon/6137511

@octplane
Copy link

octplane commented Sep 3, 2013

Nice !

@nodiscc
Copy link

nodiscc commented Sep 3, 2013

This could provide a good base for a Firefox addon that syncs shaarli to the local browser bookmarks.

@Phyks
Copy link

Phyks commented Dec 8, 2013

+1, I'd like to interface my shaarli with some other services and I definitely need something like this :)

@Phyks
Copy link

Phyks commented Jun 21, 2014

@leblanc-simon : This API is really nice. Although, I have two ideas to enhance it:

  • The token should appear explicitly somewhere in the interface, in the options for example. For now, it is not clear which value is what, and to understand how to get the token.
  • You should remove the empty test in checkParams, especially for the token. I wrote a python script using a config file, and in the default config file there was no token specified. I was trying to post new links, but my script did not fail because I did not have any token (I expected it to fail, but the request had a 200 status code because it was returning the search results).

@Phyks
Copy link

Phyks commented Jun 21, 2014

Note : Note that the position of the arguments in the request actually matters. At least, first one has to be do=api.

@nodiscc
Copy link

nodiscc commented Nov 8, 2014

@leblanc-simon @octplane @Phyks Followup to this discussion here: shaarli#16

Your comments are welcome.

By the way this is a large change and adds a lot of complexity to the current code, and it was agreed that Shaarli's core code should be KISS. So this needs to be implemented as a a plugin (proposal/discussion: shaarli#14) or we should use RSS/POST resquests as suggested in shaarli#16.

@virtualtam
Copy link

REST API on the Shaarli community fork:

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

5 participants