Plots2: Déplacer l'assistant comment_render_body dans le modèle de commentaire

Créé le 1 oct. 2018 · 4Commentaires · Source: publiclab/plots2

Cela a été marqué comme un bon candidat pour devenir un problème réservé aux débutants comme celui- ci , ce qui signifie qu'il est simple, autonome et avec un formatage supplémentaire, pourrait être un excellent point d'entrée pour un nouveau contributeur. Si vous connaissez suffisamment ce code, envisagez de le reformater ou de le republier en tant que problème réservé aux débutants , puis envoyez un ping à @publiclab/reviewers pour qu'il soit étiqueté. Ou, si ce n'est pas votre première fois, essayez de le résoudre vous-même !

https://github.com/publiclab/plots2/blob/6595ef39f25ca4f2d377cacfd92e86a7460214d6/app/helpers/application_helper.rb#L100 -L107

Faisons-en une méthode de commentaire - dans https://github.com/publiclab/plots2/blob/master/app/models/comment.rb - comme :

def render_body

end

Et supprimez-le de application_helper.rb !

Nous devrons également modifier cette ligne :

https://github.com/publiclab/plots2/blob/6595ef39f25ca4f2d377cacfd92e86a7460214d6/app/views/notes/_comment.html.erb#L120

À comment.render_body

Enfin, le test sanitize comment body dans https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rb pourrait être mis à jour pour tester comment.render_body au lieu de reproduire le code de l'aide.

Heureux d'aider avec celui-ci ! Peut-être un problème pour les débutants !

Ruby fto-candidate help wanted refactorization

Source

jywarren

Commentaire le plus utile

J'aimerais aborder cette question, ce serait l'une de mes premières contributions open source. Y a-t-il également quelque chose que j'aurais besoin de savoir avant d'ouvrir une demande d'extraction, comme un schéma spécial pour les messages de validation ?

mauricetmeyer le 1 oct. 2018

🎉1 👍1

Tous les 4 commentaires

mauricetmeyer le 1 oct. 2018

🎉1 👍1

Non, il y a un modèle dans le formulaire de demande de tirage, mais il est toujours modifiable plus tard, nous pouvons donc vous aider au fur et à mesure ! Bienvenue! 🎉

jywarren le 1 oct. 2018

👍1

@jywarren Merci ! De plus, ces problèmes de première minuterie sont parfaits pour apprendre à connaître la base de code, car je n'ai souvent pas beaucoup de temps pour lire dans de grandes bases de code. ??

mauricetmeyer le 2 oct. 2018

trop contente qu'ils te plaisent !!!

Le lun. 1 oct. 2018 à 18:09 Maurice T. Meyer [email protected]
a écrit:

@jywarren https://github.com/jywarren Merci ! Aussi ceux qui sont débutants
les problèmes sont formidables pour apprendre à connaître la base de code, car je n'ai souvent pas grand-chose
temps de lire dans de grandes bases de code. ??
-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/3552#issuecomment-426080866 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJ7XhfzVY7CxWNvf58mMmOiKBNZjlks5ugpKSgaJpZM4XCx6-
.