ページ上部にログインリンクを追加

[sourei]

トップページを価格一覧にしました。とりあえずのところですが・・・ deviseと競合するのでresources :usersをコメントにしてます。

[satomicchy]

@sourei ありがとうございます〜。ただ、画面遷移とかデザインとかは、こないだの話だと @ixkaito さんが検討することになってるんです。でも、音沙汰ないので、ちょっと待ってくださいね。Facebookのメッセージから聞いてみます。

[satomicchy]

@sourei さん、@ixkaito さんは明日から開発に入れるそうなんですけど、ページを作り始める前にとりあえず必要な画面とURLをざっと #4 に書いてみてもらえませんか？ あと、resources :usersはdeviseと競合しますか？routes確認してみてください。

[sourei]

@satomicchy さん

deviseのroutesなんですけど一部被ってしまっています。（/users(.:format)がちらほら・・・） aliaseで変更は可能なようなので少し調べてみたいと思います。

               users GET    /users(.:format)               users#index
                     POST   /users(.:format)               users#create
            new_user GET    /users/new(.:format)           users#new
           edit_user GET    /users/:id/edit(.:format)      users#edit
                user GET    /users/:id(.:format)           users#show
                     PATCH  /users/:id(.:format)           users#update
                     PUT    /users/:id(.:format)           users#update
                     DELETE /users/:id(.:format)           users#destroy
    new_user_session GET    /users/sign_in(.:format)       devise/sessions#new
        user_session POST   /users/sign_in(.:format)       devise/sessions#create
destroy_user_session DELETE /users/sign_out(.:format)      devise/sessions#destroy
       user_password POST   /users/password(.:format)      devise/passwords#create
   new_user_password GET    /users/password/new(.:format)  devise/passwords#new
  edit_user_password GET    /users/password/edit(.:format) devise/passwords#edit
                     PATCH  /users/password(.:format)      devise/passwords#update
                     PUT    /users/password(.:format)      devise/passwords#update
cancel_user_registration GET    /users/cancel(.:format)        devise/registrations#cancel
   user_registration POST   /users(.:format)               devise/registrations#create
new_user_registration GET    /users/sign_up(.:format)       devise/registrations#new
edit_user_registration GET    /users/edit(.:format)          devise/registrations#edit
                     PATCH  /users(.:format)               devise/registrations#update
                     PUT    /users(.:format)               devise/registrations#update
                     DELETE /users(.:format)               devise/registrations#destroy

[sourei]

@satomicchy さん userとのuri？競合を回避するためにdeviseの基本uriをdevise_user/***に変更しました。それと、sign_up, editにnameフィールド追加しました。 @ixkaito さん ログイン関連の画面カスタマイズ用にdeviseのビューを追加しています。

[satomicchy]

@sourei さん、ありがとうございます。いくつか質問させてください。

1. devise_parameter_sanitizer.for(:account_update) << :name は、どうして追加しましたか？
2. deviseのviewをusersの下に展開していますが、deviseはデフォルトでメール諸々(今は使ってないですが)をapp/views/devises以下のフォルダを見に行くようになっていたと思います。views/usersの下に移動した理由はなんですか？
3. routesのhelperとdevise用のform_forのurlが異なっていますが、なぜか今はPOSTできていますね。気持ち悪いのでhelperに合わせて書き換えませんか？
4. logged in したあとのプロフィール変更は、普通にuserコントローラーのeditでいいと思うのですが、どうしてdeiviseを介したんですか？
5. Log inの実装はまだですね？
6. gitのコミットは、機能毎に分けてもらえると確認がしやすくてありがたいです。

[sourei]

@satomicchy さん

1. devise_parameter_sanitizer.for(:account_update) << :name は、どうして追加しましたか？ -> ログイン後のプロファイル変更でもnameを参照する必要があったので追加しました。
2. deviseのviewをusersの下に展開していますが、deviseはデフォルトでメール諸々(今は使ってないですが)をapp/views/devises以下のフォルダを見に行くようになっていたと思います。views/usersの下に移動した理由はなんですか？ -> userモデルを共用しているためです。 rails generate devise:views users で追加したので、移動したわけではないです・・・
3. routesのhelperとdevise用のform_forのurlが異なっていますが、なぜか今はPOSTできていますね。気持ち悪いのでhelperに合わせて書き換えませんか？ -> すみません、もう少し詳しく教えてください。
4. logged in したあとのプロフィール変更は、普通にuserコントローラーのeditでいいと思うのですが、どうしてdeiviseを介したんですか？ -> ここは特に意味ないです。リンク差し替えるだけなので変更できます。
5. Log inの実装はまだですね？ -> はい、controllerでの実装はまだです。

コミットの単位気をつけますー。

[satomicchy]

@sourei さん、 なんだかdevise研究みたいになってる気がしますw。いま、githubのdeviseのREADMEを精読しながら、メソッドの動きをちょこちょこ試しています。明日時間があったら、deviseのコードリーティングとかもできたらいいなと思っています。

[sourei]

長くなったので一旦閉じます。