Plots2: Mova o auxiliar comment_render_body para o modelo de comentário
Isso foi marcado como um bom candidato para se tornar um problema exclusivo para iniciantes como este , o que significa que é simples, independente e com alguma formatação extra, pode ser um ótimo ponto de entrada para um novo contribuidor. Se você estiver familiarizado o suficiente com este código, considere reformatá-lo ou reenviá-lo como um problema exclusivo para iniciantes e , em seguida, execute ping em @ publiclab / revisores para rotulá-lo. Ou, se esta não for sua primeira vez, tente resolver você mesmo!
Vamos fazer disso um método de Comentário - em https://github.com/publiclab/plots2/blob/master/app/models/comment.rb - como:
def render_body
end
E remova-o de application_helper.rb !
Também precisaremos alterar esta linha:
Para comment.render_body
Finalmente, o teste sanitize comment body em https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rb poderia ser atualizado para testar comment.render_body vez de reproduzir o código do ajudante.
Fico feliz em ajudar com este! Pode ser um problema dos novatos!
jywarren
Todos 4 comentários
Eu gostaria de abordar esta questão, seria uma das minhas primeiras contribuições de código aberto. Além disso, há algo que eu preciso saber antes de abrir uma solicitação de pull, como um esquema especial para mensagens de confirmação?
mauricetmeyer
em 1 out. 2018
Não, há um modelo no formulário de solicitação de pull, mas ele sempre pode ser editado posteriormente, portanto, podemos ajudá-lo no processo! Receber! 👍 🎉
jywarren
em 1 out. 2018
@jywarren Obrigado! Além disso, esses problemas de primeiro cronômetro são ótimos para conhecer a base de código, já que geralmente não tenho muito tempo para ler em bases de código grandes. 👍
mauricetmeyer
em 2 out. 2018
que bom que você gostou deles !!!
Na segunda-feira, 1º de outubro de 2018 às 18h09 Maurice T. Meyer [email protected]
escreveu:
@jywarren https://github.com/jywarren Obrigado! Também aqueles primeiros
problemas são ótimos para conhecer a base de código, já que geralmente não tenho muito
hora de ler em grandes bases de código. 👍-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/publiclab/plots2/issues/3552#issuecomment-426080866 ,
ou silenciar o tópico
https://github.com/notifications/unsubscribe-auth/AABfJ7XhfzVY7CxWNvf58mMmOiKBNZjlks5ugpKSgaJpZM4XCx6-
.
jywarren
em 2 out. 2018
Questões relacionadas
keshavsethi
·
3Comentários
bronwen9
·
3Comentários
noi5e
·
3Comentários
grvsachdeva
·
3Comentários
shapironick
·
3Comentários
Comentários muito úteis
Eu gostaria de abordar esta questão, seria uma das minhas primeiras contribuições de código aberto. Além disso, há algo que eu preciso saber antes de abrir uma solicitação de pull, como um esquema especial para mensagens de confirmação?