Skip to content

add Curl support via configuration page#114

Open
Mickael-Martin wants to merge 4 commits intosebsauvage:masterfrom
Mickael-Martin:master
Open

add Curl support via configuration page#114
Mickael-Martin wants to merge 4 commits intosebsauvage:masterfrom
Mickael-Martin:master

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