Plots2: comment_render_bodyヘルパーをコメントモデルに移動します

作成日 2018年10月01日  ·  4コメント  ·  ソース: publiclab/plots2

これはなるための良い候補としてマークされている最初のタイマーだけの問題のようなこれらのことのシンプルな、自己完結型、およびいくつかの余分なフォーマットで、新しい貢献者のための偉大なエントリポイントとなることができることを意味し、。 このコードに精通している場合は、初めての問題として再フォーマットまたは再投稿してから、@ publiclab / reviewersにpingを実行してラベルを付けることを検討してください。 または、これが初めてではない場合は、自分で解決してみてください。


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

これをコメントのメソッドにしましょう-https

def render_body

end

そしてそれをapplication_helper.rbから削除してください!

この行も変更する必要があります。

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

comment.render_body

最後に、 https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rbsanitize comment bodyテストを更新して、コードを再現する代わりにcomment.render_bodyをテストすることができます。ヘルパーから。

これを手伝ってくれてうれしいです! 初めての問題かもしれません!

Ruby fto-candidate help wanted refactorization

最も参考になるコメント

私はこの問題に取り組みたいと思います。それは私の最初のオープンソースの貢献の1つになるでしょう。 また、コミットメッセージの特別なスキーマなど、プルリクエストを開く前に知っておく必要があることはありますか?

全てのコメント4件

私はこの問題に取り組みたいと思います。それは私の最初のオープンソースの貢献の1つになるでしょう。 また、コミットメッセージの特別なスキーマなど、プルリクエストを開く前に知っておく必要があることはありますか?

いいえ、プルリクエストフォームにはテンプレートがありますが、後でいつでも編集できるので、いつでもお手伝いできます。 いらっしゃいませ! 👍🎉

@jywarrenありがとうございます! また、これらの最初のタイマーの問題は、コードベースを理解するのに最適です。大きなコードベースを読み取る時間があまりないことがよくあるからです。 👍

あなたがそれらを好きでとてもうれしいです!!!

2018年10月1日月曜日午後6時9分モーリスT.マイヤー[email protected]
書きました:

@jywarren https://github.com/jywarrenありがとうございます! また、それらの最初のタイマー
私は多くの場合あまり持っていないので、問題はコードベースを知るのに最適です
大規模なコードベースを読み取る時間。 👍


あなたが言及されたのであなたはこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/publiclab/plots2/issues/3552#issuecomment-426080866
またはスレッドをミュートします
https://github.com/notifications/unsubscribe-auth/AABfJ7XhfzVY7CxWNvf58mMmOiKBNZjlks5ugpKSgaJpZM4XCx6-

このページは役に立ちましたか?
0 / 5 - 0 評価

関連する問題

grvsachdeva picture grvsachdeva  ·  3コメント

grvsachdeva picture grvsachdeva  ·  3コメント

RuthNjeri picture RuthNjeri  ·  3コメント

keshavsethi picture keshavsethi  ·  3コメント

first-timers[bot] picture first-timers[bot]  ·  3コメント