8/7 Railsポートフォリオレビュー会
スパルタコースでは毎週何かしらのイベントを開催しています。
今回もまた「Railsポートフォリオレビュー会」です。
週に一人くらいまでなら無償でレビューできると思うのでお気軽にお声がけください。
動画
【定期勉強会】 8/7 Railsポートフォリオレビュー会 | TechEssentials
雑なメモですが、こんなフィードバックをしました。
8/ 7 ポートフォリオレビュー会
githubのリポジトリ
https://github.com/satoshitodaka/rhohi2
オンラインエディタで確認できるやつ
https://github1s.com/satoshitodaka/rhohi2
サイトのURL
だいそん
サービス面
- 出発日時, 帰着日時, 作業完了日時のバリデーションが甘い
- 申請ステータス, 承認ステータスのtrue, false表記がわかりづらい
技術面
before_validation :cast_fare
def cast_fare
self.fare = # 数値に変換する処理 全角３０００ → 半角3000, あ → ???
end- rhohi2 ← ちょいとださい
- Ruby2.6.6 rubyバージョンちょい古い
- Ruby on Rails5.2.5 ちょい古い
- いらないファイルは消しましょう
- references型を使う https://github.com/satoshitodaka/rhohi2/blob/f0cd4dd083dc661ecec5fcb73b3e3661a06b3bc5/db/migrate/20210503074922_create_approvals.rb#L4
- 全体的にNOT NULL制約などの制約の制御が甘い
- URLにindexはおかしい。ってのとresourcesを使う意識を持った方が良い。 https://github.com/satoshitodaka/rhohi2/blob/f0cd4dd083dc661ecec5fcb73b3e3661a06b3bc5/config/routes.rb#L6
resources :approvals do get :approved, on: :collection end - 招待 invitationsテーブル
- invitations
- id
- name
- token(aiudfwae8fpaqe8oifq8798ew7r)
- メールには「こちら(http://localhost:3000/users/new?token=aiudfwae8fpaqe8oifq8798ew7r)から登録お願いします」
- ユーザーがユーザを登録する（invitationsからemail, nameを取得して自動的に入力しておく。パスワードとパスワード確認をユーザー自身が入力する。で、登録）
- invitations
- adminはadmin用のnamespaceを切った方が何かと制御しやすい
- aprovals/aproveds_controllers#index ← こっちで制御するとDHHっぽい。https://github.com/satoshitodaka/rhohi2/blob/f0cd4dd083dc661ecec5fcb73b3e3661a06b3bc5/app/controllers/approvals_controller.rb#L13
- https://tech.kitchhike.com/entry/2017/03/07/190739
- https://github.com/satoshitodaka/rhohi2/blob/f0cd4dd083dc661ecec5fcb73b3e3661a06b3bc5/test/integration/approver_action_test.rb#L25
@other_users_statement.reload.approved
ゆう
サービス面
- 旅費手当新規登録画面で入力に不備がある場合、画面を進めると入力内容が消えた。入力必須項目の表示と、入力に不備があった際も内容の引き継ぎがあると良いと思いました。
- 料金を打ち込みではなく、選択式のものにしたらより良いと思いました。
- 出張登録申請した後、編集できません。
- サイトの使い方の説明ページが欲しいと思いました。(aboutページがあるので後回しにされてると思うのですが・・）
- とはいえ説明がなくてもサイトが使えそうで、機能が必要なものだけでサイトが作られてると感じて良いなと思いました。
その他
https://github.com/satour/rails-style-guide-jp
こんな感じで毎週勉強会をやってます。
スパルタコースはサポートの質が落ちないように少数のコミュニティとして活動しています。
実際に未経験からRailsの業務委託で仕事をしている人もいるくらいなのでちゃんと学習すればスキルはつきます。
普段はSlackでやりとりしてますのでリンクを貼っておきますね。
Slackコミュニティへの招待リンク
ではでは。
