search

yosuke-43 / dog_app_old

1 stars 0 forks source link

MVPリリースレビュー #93

Open kenchasonakai opened 9 months ago

kenchasonakai commented 9 months ago

MVPリリースおめでとうございます！ このissueのコメントに本リリース予定日の返答をお願いします 以下にレビュー内容を記載します

総評

コメント機能のAjax機能やタイトル&本文のマルチカラム検索など頑張っているところはある一方で、技術のアピールとしてはもう一歩という印象を受けました。 下記のレビューを参考にステップフォームの導入や画像加工などを行なったり、rails_ujsを使ったAjaxではなくturboを使った非同期処理を使ったり、DockerやGitHubActionsを使ったCI/CDを導入するなどしてアピール出来るポイントを作っていくとよりよくなると思います。

ユーザー登録・ログイン機能

ユーザー登録後、ログイン画面に遷移するのではなくオートログイン出来ると楽かなと思いました

診断機能

項目が多いのでステップフォームを採用したりすると使いやすくなるかなと思いました

投稿表示

ログインしないと診断後導線の犬種ごとの投稿が見れないので特に理由がなければログイン前にも見れると良いかなと思います

投稿作成

投稿時の画像プレビューがエラー出ていると思うので修正しましょう

画像

画像が全体的に崩れてしまっているのでCarrierwaveやminimagickなどの画像加工出来るものでトリミングしたりして画像が崩れないように工夫すると良いかと思います

コードの管理

いらないファイル(development.sqlite3とか使ってないdecoratorとか)がいっぱいあるのでgit管理しないようにすると良いと思います データ入れ込みのためのSQLファイルがrootディレクトリにあるのでそれ用のディレクトリを作成してまとめてあげると綺麗に見えると思います vendorにgemが入ったままgit管理されてるのでgit管理から外しましょう git commitする前に本当にcommitすべきファイルなのかどうか確認して、git管理が必要ないファイルはgitignoreに入れましょう

yosuke-43 commented 9 months ago

@kenchasonakai レビューしていただきありがとうございます。 3月7日(木)に本リリース予定にしたいと思います。

kenchasonakai commented 9 months ago

ありがとうございます！ 承知いたしました 🙆‍♂️

yosuke-43 commented 9 months ago

MVPリリースのレビューを受けての修正点