RemitRequest モデルの分割

0gajun / 2018-newbies

1 stars 1 forks source link

Closed 0gajun closed 6 years ago

0gajun commented 6 years ago

RemitRequestモデルの分割の為に

を行った。

Rspecの書き方はよくわからんので、テストケースだけのレビューにしたさ

0gajun commented 6 years ago

https://moneyforward.slack.com/archives/CAL369FCN/p1525850907000411

ここの議論でtinyint使うように治そうと思ったが、ちょっと ActiveRecord::Enum 使うように変更するのシンドいので、一旦このまま進もうと思います...

サービスインする前に直せれば嬉しい

ryoff commented 6 years ago

時間がかぎられる研修だから、そういう判断もありだと思う 👍

そしてそういう判断はログが残るように、githubやcommit messageに残せるといいね 👍

ryoff commented 6 years ago

スペースやインデントずれが何箇所かあったので、rubocopの自動修正走らせておくといいかもー 🙋

0gajun commented 6 years ago

スペースやインデントずれが何箇所かあったので、rubocopの自動修正走らせておくといいかもー

走らせました 🙆

全部に走らせると色々変更混ざっちゃいそうなので、変更ファイルだけ 👍

ShuheiGoto commented 6 years ago

ざっと見た感じ良さそうな気がする specらへんすごい...!