Add wallabag support #63
Conversation
|
I'm ok for merging, but wait for confirmation from @Phyks since he wrote the file that this PR modifies. |
| @@ -57,6 +57,6 @@ | |||
| </article> | |||
| {/loop} | |||
|
|
|||
| <p>{$nb_pages} pages</p> | |||
There was a problem hiding this comment.
Un test est mieux ici, pour directement afficher l'accord correct. C'était sur ma TODO-list, tant qu'à modifier cette ligne, autant le faire.
|
@qwertygc : J'ai ajouté quelques annotations. Ok pour merger si corrigé. |
|
J'ai corrigé ces petits problèmes. |
|
@qwertygc : Tu as un wallabag et tu as testé le code de partage ? J'ai mergé la PR, mais je n'ai pas de wallabag à disposition pour tester. |
|
J'ai testé le code dans d'autres outils, donc oui ca marche. Et sur freeder aussi, du moins chez moi. On 21 septembre 2014 13:36:01 UTC+02:00, Lucas Verney notifications@github.com wrote:
Envoyé de mon téléphone Android avec K-9 Mail. Excusez la brièveté. |
|
Ok, nickel ! |
|
Bon, j'ai corrigé, et y'avait un petit bug (c'est du quasi mystique car je me suis basé sur du booklmarklet. Mais bon, comme ça ca marche) |
Add wallabag support...