Plots2: Kommentar_render_body-Helfer in Kommentarmodell verschieben
Dies wurde als guter Kandidat für eine solche Ausgabe nur für Erstanfänger markiert, was bedeutet, dass es einfach, in sich geschlossen und mit etwas zusätzlicher Formatierung ein großartiger Einstiegspunkt für einen neuen Beitragenden sein könnte. Wenn Sie mit diesem Code vertraut sind, ziehen Sie in Betracht, ihn neu zu formatieren oder als Erstausgabe neu zu veröffentlichen , und
Machen wir dies zu einer Kommentarmethode - in https://github.com/publiclab/plots2/blob/master/app/models/comment.rb - wie:
def render_body
end
Und entferne es aus application_helper.rb !
Wir müssen auch diese Zeile ändern:
Zu comment.render_body
Schließlich könnte der sanitize comment body Test in https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rb aktualisiert werden, um comment.render_body zu testen, anstatt den Code zu reproduzieren vom Helfer.
Gerne helfe ich bei diesem! Könnte ein Anfängerproblem sein!
jywarren
Alle 4 Kommentare
Ich möchte mich diesem Thema annehmen, es wäre einer meiner ersten Open-Source-Beiträge. Gibt es auch etwas, das ich vor dem Öffnen eines Pull-Requests wissen muss, wie ein spezielles Schema für Commit-Nachrichten?
mauricetmeyer
am 1. Okt. 2018
Nein, es gibt eine Vorlage im Pull-Request-Formular, die aber später immer bearbeitet werden kann, damit wir Ihnen dabei helfen können! Willkommen! 👍 🎉
jywarren
am 1. Okt. 2018
@jywarren Danke! Auch diese ersten Probleme sind großartig, um die Codebasis kennenzulernen, da ich oft nicht viel Zeit habe, um in große Codebasen einzulesen. 👍
mauricetmeyer
am 2. Okt. 2018
schön dass sie dir gefallen!!!
Am Mo, 1. Okt 2018 um 18:09 Maurice T. Meyer [email protected]
schrieb:
@jywarren https://github.com/jywarren Danke! Auch die Erstbesucher
Probleme sind toll, um die Codebasis kennenzulernen, da ich oft nicht viel habe
Zeit, in große Codebasen einzulesen. 👍—
Sie erhalten dies, weil Sie erwähnt wurden.
Antworten Sie direkt auf diese E-Mail und zeigen Sie sie auf GitHub an
https://github.com/publiclab/plots2/issues/3552#issuecomment-426080866 ,
oder den Thread stumm schalten
https://github.com/notifications/unsubscribe-auth/AABfJ7XhfzVY7CxWNvf58mMmOiKBNZjlks5ugpKSgaJpZM4XCx6-
.
jywarren
am 2. Okt. 2018
Verwandte Themen
![first-timers[bot] picture](https://avatars.githubusercontent.com/in/4832?v=4&s=40)
first-timers[bot]
·
3Kommentare
keshavsethi
·
3Kommentare
noi5e
·
3Kommentare
jywarren
·
3Kommentare
grvsachdeva
·
3Kommentare
Hilfreichster Kommentar
Ich möchte mich diesem Thema annehmen, es wäre einer meiner ersten Open-Source-Beiträge. Gibt es auch etwas, das ich vor dem Öffnen eines Pull-Requests wissen muss, wie ein spezielles Schema für Commit-Nachrichten?