# 8/ 7 ポートフォリオレビュー会
### githubのリポジトリ

https://github.com/satoshitodaka/rhohi2

オンラインエディタで確認できるやつ
https://github1s.com/satoshitodaka/rhohi2

### サイトのURL
https://rhohi.herokuapp.com/

# だいそん
## サービス面
- 出発日時, 帰着日時, 作業完了日時のバリデーションが甘い
- 申請ステータス, 承認ステータスのtrue, false表記がわかりづらい

## 技術面
- 運賃の全角対応
- https://github.com/satoshitodaka/rhohi2/blob/f0cd4dd083dc661ecec5fcb73b3e3661a06b3bc5/app/models/expence.rb#L10

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
- email
- name
- token(aiudfwae8fpaqe8oifq8798ew7r)
- メールには「こちら(http://localhost:3000/users/new?token=aiudfwae8fpaqe8oifq8798ew7r)から登録お願いします」
- ユーザーがユーザを登録する（invitationsからemail, nameを取得して自動的に入力しておく。パスワードとパスワード確認をユーザー自身が入力する。で、登録）
- 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`

# ○○○
## サービス面
- xxx
- xxx

## 技術面
- xxx
- xxx

# ゆう
## サービス面
- 旅費手当新規登録画面で入力に不備がある場合、画面を進めると入力内容が消えた。入力必須項目の表示と、入力に不備があった際も内容の引き継ぎがあると良いと思いました。
- 料金を打ち込みではなく、選択式のものにしたらより良いと思いました。
- 出張登録申請した後、編集できません。
- サイトの使い方の説明ページが欲しいと思いました。(aboutページがあるので後回しにされてると思うのですが・・）
- とはいえ説明がなくてもサイトが使えそうで、機能が必要なものだけでサイトが作られてると感じて良いなと思いました。

# その他
https://github.com/satour/rails-style-guide-jp