readthedocs-fr / bin-server

Un outil pour héberger des snippets de code et les partager via une URL.
https://bin.readthedocs.fr
MIT License
14 stars 15 forks source link

ref: snippet revision plus global cleanup #80

Closed Julien00859 closed 3 years ago

Julien00859 commented 3 years ago

Add the missing snippet revision feature, users can duplicate existing snippets to edit them. The new snippet links its parent so it is easy to compare them.

The GET / URL has been updated, it takes two optionnal query parameters: (i) lang the language by default selected, it override the config default. (ii) parentid the snippet to copy the code from.

The POST /new URL has been updated, it takes a form input parentid, the original snippet the new one is a revision.

Additionnaly to the above feature, the source code have been scanned and various minor changes have been made. In any order:

While they are not needed by themselves, it is a bit of house keeping that is sure to be welcome by new contributers.

Co-authored-by: Mestery 48163546+Mesteery@users.noreply.github.com Co-authored-by: wyx0-xyz wyxo.bs09@gmail.com

Closes #10

Julien00859 commented 3 years ago

Les boutons sont dégueux et il faut encore écrire les tests unitaires, qui s'y colle ? (shotgun pas moi)

Mesteery commented 3 years ago

Les boutons sont dégueux et il faut encore écrire les tests unitaires, qui s'y colle ? (shotgun pas moi)

Je m'en occuperai

Nauhai commented 3 years ago

C'est pas toi qui m'avait dit qu'il fallait jamais laisser un truc "à faire" sur une Pull Request ? :derp:

Mesteery commented 3 years ago

Si, il y est, mais il est peu visible. Mais je suis en train de m'en occuper.

Nauhai commented 3 years ago

Aussi petit bug mais pas très important, quand on clique sur "Voir le parent", ça amène sur /<parentid>?, avec le ?.

Mesteery commented 3 years ago

Aussi petit bug mais pas très important, quand on clique sur "Voir le parent", ça amène sur /<parentid>?, avec le ?.

Oui, c'est normal, étant donné qu'un form (methode GET) est utilisé, le navigateur met ? pour les potentiels params. Ceci sera remplacé par un simple a.

Mesteery commented 3 years ago

J'ai essayé d'ajouter des tests unitaires mais j'ai pas réussi.

Mesteery commented 3 years ago

image Respectivement :

Mesteery commented 3 years ago

je ne me vois pas l'utiliser pour voir la révision d'avant.

Certes, mais l'icone reste plus approprié pour "Voir la revision precedente" que "Modifier"

Julien00859 commented 3 years ago

Pour l'icone snippet parent, je veux une flèche vers la gauche, mieux si c'est une flèche vers la gauche dans un document