Plots2: Mover el ayudante comment_render_body al modelo de comentarios
Esto ha sido marcado como un buen candidato para convertirse en un tema exclusivo para este , lo que significa que es simple, autónomo y, con un formato adicional, podría ser un excelente punto de entrada para un nuevo colaborador. Si está lo suficientemente familiarizado con este código, considere reformatearlo o volver a
Hagamos de esto un método de comentario, en https://github.com/publiclab/plots2/blob/master/app/models/comment.rb , como:
def render_body
end
¡Y elimínelo de application_helper.rb !
También necesitaremos cambiar esta línea:
Hasta comment.render_body
Finalmente, la prueba sanitize comment body en https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rb podría actualizarse para probar comment.render_body lugar de reproducir el código del ayudante.
¡Encantado de ayudar con este! ¡Podría ser un problema para los principiantes!
jywarren
Todos 4 comentarios
Me gustaría abordar este tema, sería una de mis primeras contribuciones de código abierto. Además, ¿hay algo que deba saber antes de abrir una solicitud de extracción, como un esquema especial para los mensajes de confirmación?
mauricetmeyer
en 1 oct. 2018
No, hay una plantilla en el formulario de solicitud de extracción, pero siempre se puede editar más adelante, ¡así que podemos ayudarlo a medida que avanza! ¡Bienvenido! 👍 🎉
jywarren
en 1 oct. 2018
@jywarren ¡ Gracias! Además, esos problemas por primera vez son excelentes para conocer la base de código, ya que a menudo no tengo mucho tiempo para leer bases de código grandes. 👍
mauricetmeyer
en 2 oct. 2018
¡¡¡Me alegro de que te gusten !!!
El lunes 1 de octubre de 2018 a las 6:09 p.m. Maurice T. Meyer [email protected]
escribió:
@jywarren https://github.com/jywarren ¡ Gracias! También los novatos
Los problemas son geniales para conocer el código base, ya que a menudo no tengo mucho
tiempo para leer en grandes bases de código. 👍-
Recibes esto porque te mencionaron.
Responda a este correo electrónico directamente, véalo en GitHub
https://github.com/publiclab/plots2/issues/3552#issuecomment-426080866 ,
o silenciar el hilo
https://github.com/notifications/unsubscribe-auth/AABfJ7XhfzVY7CxWNvf58mMmOiKBNZjlks5ugpKSgaJpZM4XCx6-
.
jywarren
en 2 oct. 2018
Temas relacionados
milaaraujo
·
3Comentarios
divyabaid16
·
3Comentarios
jywarren
·
3Comentarios
ebarry
·
3Comentarios
![first-timers[bot] picture](https://avatars.githubusercontent.com/in/4832?v=4&s=40)
first-timers[bot]
·
3Comentarios
Comentario más útil
Me gustaría abordar este tema, sería una de mis primeras contribuciones de código abierto. Además, ¿hay algo que deba saber antes de abrir una solicitud de extracción, como un esquema especial para los mensajes de confirmación?