Skip to content

Answer by Shodai - #25

Open
imadashodai wants to merge 2 commits into
JunichiIto:mainfrom
imadashodai:myanswer
Open

Answer by Shodai#25
imadashodai wants to merge 2 commits into
JunichiIto:mainfrom
imadashodai:myanswer

Conversation

@imadashodai

Copy link
Copy Markdown

コードについて

  • コントローラーのメソッドが大きくなりすぎないように気をつけました。
  • モデル内のメソッドを細分化してもいいかなと思いました。

レビューお願いいたします。

@JunichiIto

Copy link
Copy Markdown
Owner

すでにチェック済みかもしれませんが、こちらの動画でコメントさせてもらいました!
よろしくお願いします。

【前編】全部見ます!プロを目指すRailsエンジニアのための公開コードレビュー #railsdm - YouTube
https://www.youtube.com/watch?v=xZk4f4PPUhU&t=1s

【後編】全部見ます!プロを目指すRailsエンジニアのための公開コードレビュー #railsdm - YouTube
https://www.youtube.com/watch?v=yDxg0sXHXOo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants