search

okjm3324 / myapp

0 stars 0 forks source link

MVPレビュー #73

Closed Tsuchiya2 closed 11 months ago

Tsuchiya2 commented 12 months ago

MVPリリースお疲れ様でした、下記の視点をもとにレビューいたします。

  1. READMEから課題解決する実装がされているか&動いている
  2. 静的コードチェックが入っている
  3. 就活において十分な技術力を示せる機能を実装しているか
  4. UIでユーザーが迷わないような導線を作れているか・ユーザーが使いたくなるようなサービス設計ができているか

READMEから課題解決する実装がされているか&動いている

音声の再生(AB区間)できるのも確認しました。LGTMです。

静的コードチェックが入っている

・Rubocopを通しましょう。アクションで何も書かれていないものは1行にするようにとか指摘が出るかと思われます。 ・使用していないファイルは削除しておきましょう。（viewファイルでそれらしきものが散見されました） ・viewファイルのインデントが半角4つになっているものがありました。半角2つに修正しましょう。(_header.html.erb) ・tracks/edit.html.erbの Githubissues.

  • Githubissues is a development platform for aggregating issues.