search

tkm201113 / sanpo-de-gourmet

0 stars 0 forks source link

"Issueベース開発"レビュー依頼 #26

Open tkm201113 opened 2 weeks ago

tkm201113 commented 2 weeks ago

実装予定の機能

必要なページ

ky0613 commented 2 weeks ago

対応ありがとうございます。 以下コメントです。

やや粒度が大きいためか、または見通しが難しいためか、ストーリーポイントが全体的に少し大きめに感じました。ポイント1, 2 くらいになるくらいに細かくissue化することで、不明瞭な部分、明瞭な部分が可視化されますし、進捗を感じながら開発を進めることができます。もう少しissueを細分化しましょう。

tmtkzk0823 commented 2 weeks ago

山口講師のコメントに対しての返答がない状態で、再提出をいただいているのですが、修正したかどうかがコメントで残されていないため、一度戻させていただきますね。

issueや、プルリクエストではテキストのやり取りが基本となります。 修正後は、レビューを受けた講師のコメントに対して「どのような修正を行なったのか」を簡単でもいいのでコメントし、返答するように意識しましょう。 上記の対応で、別のレビュワーが見た時に、どのようなやり取りをしたかが履歴で終えるようになります。 よろしくお願いします。

tkm201113 commented 2 weeks ago

ky0613様 tmtkzk0823様

コメントありがとうございます。

issueですが、ストーリーポイントが「1」または「2」になるように細分化しました。 お手数おかけしますが、ご確認のほどよろしくお願いいたします。

kashiwagi-rena commented 2 weeks ago

お疲れ様です 下記、検討をお願いいたします

  1. Google Map追加こちらのissueが粒が大きいと思います。ざっくりでいいので、記事等をみて、もう少し小さくしてみてください。また、みた記事はissueに貼り付けておくと良いです。
  2. ヘッダーとフッターのレイアウトの追加とそれに伴った各レイアウトの重複部分の削除をお願いいたします。（Reactを使用されるとのこと、コンポーネントを調べて頂くとレイアウトのissueがスッキリするかと思います。参考までに）
tkm201113 commented 2 weeks ago

kashiwagi-rena様 コメントありがとうございます。

以下の項目を修正しましたので、ご確認お願いいたします。

  1. 「Google Map追加」の細分化
  2. ヘッダーとフッターのレイアウトの追加と、それに伴う各レイアウトの重複部分の削除
  3. その他、可能な部分の細分化
kashiwagi-rena commented 2 weeks ago

お疲れ様です。 こちら、ユーザー登録機能並びに認証をどのgemで行うかの記載はされてますでしょうか？ されていなければご確認の上追記お願いいたします。

tkm201113 commented 2 weeks ago

kashiwagi-rena様 コメントありがとうございます。

ユーザー登録機能並びに認証をどのgemで行うのかを追記しました。 お手数おかけしますが、ご確認のほどよろしくお願いいたします。

Tsuchiya2 commented 2 weeks ago

概ね良いかと思います。LGTMです。

以下、MVPに含めなくて良いと思ったものになります。