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 Curl support via configuration page #114

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

add Curl support via configuration page #114

wants to merge 4 commits into from

Conversation

Mickael-Martin
Copy link

Il y a peut-être quelques trucs à modifier, mais la grosse partie de la gestion du Curl est là.
Maybe some thinks to modify...

@Nabellaleen
Copy link

Pourrais-tu documenter l'objectif de tes modifications ? Je crois comprendre à partir du diff' que tu veux utiliser Curl pour récupérer les info' d'un lien depuis une page HTTPS, mais je n'en suis pas sûr.

Il serait donc pratique que tu documentes dans le README l'utilisation de l'option Curl, éventuellement les pré-requis pour que ça fonctionne, etc.

Tu feras attention aussi, je crois que l'indentation de ton code n'est pas uniforme avec le reste (tabulations à la place d'espaces, ou trucs comme ça)

pikzen pushed a commit to pikzen/Shaarli that referenced this pull request Feb 18, 2015
redirect to previous search (if any) when deleting a link
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