Plots2: Pindahkan pembantu comment_render_body ke dalam model Komentar
Ini telah ditandai sebagai kandidat yang baik untuk menjadi masalah pertama kali saja seperti ini , artinya sederhana, mandiri, dan dengan beberapa pemformatan tambahan, bisa menjadi titik masuk yang bagus untuk kontributor baru. Jika Anda cukup familier dengan kode ini, harap pertimbangkan untuk memformat ulang atau memposting ulang sebagai masalah khusus pemula , lalu ping ke @publiclab/reviewers untuk memberinya label. Atau, jika ini bukan pertama kalinya Anda, coba selesaikan sendiri!
Mari jadikan ini metode Komentar - di https://github.com/publiclab/plots2/blob/master/app/models/comment.rb - seperti:
def render_body
end
Dan hapus dari application_helper.rb !
Kita juga perlu mengubah baris ini:
Untuk comment.render_body
Terakhir, pengujian sanitize comment body di https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rb dapat diperbarui untuk menguji comment.render_body alih-alih mereproduksi kode dari penolong.
Senang bisa membantu dengan yang satu ini! Bisa jadi masalah pemula!
jywarren
Semua 4 komentar
Saya ingin membahas masalah ini, ini akan menjadi salah satu kontribusi open source pertama saya. Juga adakah yang perlu saya ketahui sebelum membuka permintaan tarik, seperti skema khusus untuk pesan komit?
mauricetmeyer
pada 1 Okt 2018
Tidak, ada templat dalam formulir permintaan tarik, tetapi selalu dapat diedit nanti, jadi kami dapat membantu Anda saat Anda pergi! Selamat datang! 👍
jywarren
pada 1 Okt 2018
@jywarren Terima kasih! Juga masalah pengatur waktu pertama itu bagus untuk mengenal basis kode, karena saya sering tidak punya banyak waktu untuk membaca basis kode besar. 👍
mauricetmeyer
pada 2 Okt 2018
sangat senang Anda menyukai mereka !!!
Pada Senin, 1 Okt 2018 pukul 18:09 Maurice T. Meyer [email protected]
menulis:
@jywarren https://github.com/jywarren Terima kasih! Juga pengatur waktu pertama itu
masalah bagus untuk mengenal basis kode, karena saya sering tidak punya banyak
waktu untuk membaca ke dalam basis kode besar. 👍—
Anda menerima ini karena Anda disebutkan.
Balas email ini secara langsung, lihat di GitHub
https://github.com/publiclab/plots2/issues/3552#issuecomment-426080866 ,
atau matikan utasnya
https://github.com/notifications/unsubscribe-auth/AABfJ7XhfzVY7CxWNvf58mMmOiKBNZjlks5ugpKSgaJpZM4XCx6-
.
jywarren
pada 2 Okt 2018
Masalah terkait
divyabaid16
·
3Komentar
milaaraujo
·
3Komentar
noi5e
·
3Komentar
milaaraujo
·
3Komentar
ebarry
·
3Komentar
Komentar yang paling membantu
Saya ingin membahas masalah ini, ini akan menjadi salah satu kontribusi open source pertama saya. Juga adakah yang perlu saya ketahui sebelum membuka permintaan tarik, seperti skema khusus untuk pesan komit?