yu-croco / airbnb_clone Goto Github PK

テンプレートにstyleが直書きされている箇所があるので、scssに分離する。

ビューテンプレートにjsを直書きしている箇所があるので、jsファイルに分離する。
対応時にはwebpackerを調べること。

rails 5 の webpacker 使えば、特定のファイルだけに特定の JS 読み込むのも簡単です。
そこまでやる必要はないですが。

[Overviews]
When I got to users/sign_in or users/sign_up , I get this screen.

ご指摘元のコメント

Listing から find_by するのは少し怖いです。
他の User の Listing をまちがって取ってくる可能性があるからです。リクエスト次第では有りえます。

以下のように探してくれば、そのユーザ以外の Listing は表示されないです。

current_user.listings.find_by(...)

現状ではRailsのform_for/form_tagを使用してformタグを作成しているので、simple_formで書き換える。

現状では、Listingsコントローラに無理にアクションを入れている。
（basics、description、address、price、photos、calendar、bankaccount）
これらをListingsコントローラから切り出して、それぞれに合うコントローラを作成する。
[コメント元]

間違いではないのですが、こういった controller は RESTful の**に反していて、controller が複雑になるかなと思います。
http://postd.cc/how-dhh-organizes-his-rails-controllers/

あとは、可読性も下げる気がしています。

ListingsController#address がどういう情報を表しているのかわかりにくかったりします。
ListingAdressesController#show or #index とかにしておければ、何を表しているのかわかりやすいです。

コメント元

house_type は選択式ですよね？
これだと意図しない選択肢が DB に残る可能性があって危険です。
enumerize を使うと良いかと。

現状、issueへの記載方法が定まっていないのでおよそのフォーマットを決める。
フォーマットはwikiに記載予定。

[不具合内容]
githubへのpush時に発火するテストが正常に走らない。
（Rails 5.x系でCircle CI繋ぐと、割りと遭遇してるっぽい。？）

bundler:
failed to load command: rspec (/home/circleci/repo/vendor/bundle/ruby/2.4.0/bin/rspec)
LoadError: cannot load such file -- rspec_junit_formatter

現状でDBスキーマでnull:false対応ができていない箇所があるので、null:falseを追加する。
[対象テーブル/カラム名]

• listings/house_type
• listings/active
• photos/listing_id

[コメント元]

とくに boolean については null: false は徹底したほうがよいです。
true, false, nil, の 3 パターンになってしまって実装が面倒になります。
http://qiita.com/jnchito/items/a342b64cd998e5c4ef3d

circle ci 使うと、github 上でテスト実行してもらえるから便利ですよ！
1 つのレポジトリーだけなら無料でつかえます。
http://qiita.com/noboru_i/items/7d300eb63ae667bf8dc2