kyoshida-aim / RailsTraining

Ruby on Rails研修のPR作成・進捗管理用
https://ky-railstraining-2019.herokuapp.com/
1 stars 0 forks source link

管理画面でのユーザー登録・編集をパーシャル化 #40

Closed kyoshida-aim closed 5 years ago

kyoshida-aim commented 5 years ago

概要

理由

確認方法

ユーザー登録・編集が行えること

やっていないこと

相談事項

taskの方でフォームもパーシャル化したので今回もフォームをパーシャル化しているか、アクションによって要求されるフォームが変わるし、無理にパーシャル化する必要はなかったかも (formのsubmitのみ記述が減ったけど)

今回はエラーメッセージだけの切り出しだけで良かった?

vividmuimui commented 5 years ago

taskの方でフォームもパーシャル化したので今回もフォームをパーシャル化しているか、アクションによって要求されるフォームが変わるし、無理にパーシャル化する必要はなかったかも (formのsubmitのみ記述が減ったけど) 今回はエラーメッセージだけの切り出しだけで良かった?

そうおもいます!

vividmuimui commented 5 years ago

1フィールドだけif文によって変わる、ぐらいだったらまとめてよかったかもですが、 結構ガッツリ違うので、まとめる必要はなかった感じがしますね。