Plots2: profile.html.erbコードpt.2のリファクタリング

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

初めて?

これは初めての問題です。 これは、以前にコードベースに貢献したことがない人、または以前にオープンソースに貢献したことがない人でも読みやすくするために取り組んできたことを意味します。

それがあなたなら、私たちはあなたが最初の一歩を踏み出すのを手伝うことに興味があり、あなたがそうするように質問に答えてあなたを助けることができます。 私たちは、無料のオープンソースソフトウェアで過小評価されているグループの人々からの貢献に特に関心があることに注意してください!

プルリクエストを作成するプロセスは、新しい貢献者にとって最大の障壁であることを私たちは知っています。 この問題はあなたのためです💝

以前に貢献したことがある場合は、これを新しい人に任せて、一般的なヘルプが必要な問題を確認することを検討してください。 ありがとう!

🤔あなたが知る必要があること。

何もない。 この問題は、オープンソースへの歓迎を目的としています:)プロセスを順を追って説明します。

問題

app/views/users/profile.html.erbコードは非常に冗長です...リファクタリングを使用できます

コードから繰り返し発生する行をリファクタリングすることから始めることができます。 今のところ、次の1行目:

https://github.com/publiclab/plots2/blob/1931439c0c799b3e3420a9dcaace7a54cf895bd4/app/views/users/profile.html.erb#L216 -L223

解決

何に変更するか:

「問題」セクションにあるコードブロックの行をリファクタリングして、代わりに新しいヘルパーメソッドを使用します。

その行は<% if current_profile_user %>更新できます

ありがとう!!

修正する手順

  • []以下のコメントでこの問題を主張し、必要な明確な質問をしてください
  • []リポジトリをフォークし、メインリポジトリのREADMEの指示に従ってローカルにセットアップしますhttps://github.com/publiclab/plots2

    • []問題を説明する一意の名前で新しい機能ブランチを作成します

  • []上記の手順に従って問題を修正してみてください。ただし、完了する前でも、次のことができます。
    ブランチに変更をコミットし、プルリクエストを開始します( Public Labソフトウェアへの貢献を参照)が、質問がある場合、または終了していない場合は、「進行中」としてマークします。
  • []プルリクエスト本文でこの問題を参照してください
  • []プルリクエストを送信すると、マージするための追加のチェックリストが提供されます

💬助けを得る

ヘルプが必要な場合-ここにいくつかのオプションがあります:

first-timers-only help wanted

最も参考になるコメント

私はまだオープンソースの貢献をしていないので、これは私を暖かくします。 これを主張してもいいですか?

全てのコメント4件

私はまだオープンソースの貢献をしていないので、これは私を暖かくします。 これを主張してもいいですか?

こんにちは@ dave-light、どうぞどうぞ。 そして、パブリックラボとオープンソースワールドへようこそ:globe_with_meridians :: earth_americas:。

@ dave-light:あなたはそれに乗っていますか? またはこれを主張することができますか?

ねえ@mzmmlds 、私はそれに乗っています。

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

関連する問題

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

milaaraujo picture milaaraujo  ·  3コメント

keshavsethi picture keshavsethi  ·  3コメント

grvsachdeva picture grvsachdeva  ·  3コメント

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