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

Refactorer le code d'aide à la rédaction #3896

Open
vhf opened this issue Oct 25, 2016 · 0 comments
Open

Refactorer le code d'aide à la rédaction #3896

vhf opened this issue Oct 25, 2016 · 0 comments
Labels
C-Back Concerne le back-end Django S-Refactorisation Améliore le code existant sans forcément ajouter de nouvelle fonctionnalité

Comments

@vhf
Copy link
Contributor

vhf commented Oct 25, 2016

(Faire des tickets à propos des choses qu'on pourrait refactorer : je sais pas si ça va marcher, mais on peut toujours essayer. Ça a au moins l'avantage d'être des tickets relativement faciles et qui permettent de se familiariser avec le code. Et vu que pour le moment y'a presque jamais de PR de refactorisation, ça vaut le coup d'essayer ! :) )

Fork de cette discussion : #2872 (comment)

Voici ce qu'il faudrait refactorer à mon avis :

  • ContentsWithHelps
    • ContentsWithHelps n'est pas un super nom, c'est plus des contenus pour lesquels on a demandé de l'aide que des contenu qui ont des aides
    • Il faudrait corriger les commentaires, et notamment en enlever le renvoi vers la ZEP
    • Refactorer les strings en dur et le if/elif
    • S'assurer que le nom de la classe doit bien être pluralisé, j'arrive pas à inférer la convention utilisée
  • HelpWriting
    • HelpWriting n'est pas assez explicite : illustrer corriger ou reprendre font partie de l'édition d'un contenu, pas de son écriture, je verrais bien quelque chose comme EditorialHelp
    • HelpWriting est dans utils alors qu'elle ne semble utilisée que dans les tutos, du coup ça devrait plutôt être un modèle du module de tuto ?
    • tablelabel => table_label
@vhf vhf added Facile Bon ticket pour débuter pour rejoindre le développement ! C-Back Concerne le back-end Django S-Refactorisation Améliore le code existant sans forcément ajouter de nouvelle fonctionnalité labels Oct 25, 2016
@artragis artragis removed the Facile Bon ticket pour débuter pour rejoindre le développement ! label Apr 13, 2020
@Arnaud-D Arnaud-D moved this to À trier in Suivi des tickets Dec 17, 2022
@Arnaud-D Arnaud-D moved this from À trier to Quand les poules auront des dents in Suivi des tickets Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-Refactorisation Améliore le code existant sans forcément ajouter de nouvelle fonctionnalité
Projects
Status: Quand les poules auront des dents
Development

No branches or pull requests

2 participants