Conversation
|
テストコードの指摘事項修正をマージしても、この解答例はパスしました。 |
|
|
||
| # Reduces boot times through caching; required in config/boot.rb | ||
| gem 'bootsnap', '>= 1.1.0', require: false | ||
| gem 'bootstrap', '~> 4.3.1' |
| validates :last_name, presence: true, length: { maximum: 255 } | ||
|
|
||
| def mine?(object) | ||
| # [注意]ここでは current_user は使用できないため、idで比較する |
There was a problem hiding this comment.
確かに要らないですね、もし補足するとしたら
self.id の self を省略してmine?メソッドの呼び出し元の＠userなどのオブジェクトの id を呼び出しています
みたいな内容の方を補足した方が親切かもしれないですね🤔
There was a problem hiding this comment.
71ad84c でself.idを使っている点を補足するコメントに変更しました！
|
@DaichiSaito
|
ごめん漏れてた！！ |
|
トップページのViewも |
|
masterにheaderやfooter、アイコン画像やFlashメッセージのパーツなどをコミットして、以下の状態にしました！ |
|
Spec実行結果です |
|
諸々差分をキレイにした後のテスト実行結果 |
|
pendingって残ったままでいいんだっけ？不要なら消しちゃえば？ |
|
直しておきました！ |
gitignoreにvendorを含める
ローカル環境でもrubocop-railsを読み込むように修正
READMEにsasscのインストール手順を追記
Master convert to english
Answer convert to english
4 participants
このPRで行なっている実装を、確認テストで生徒に実施してもらうことを期待しています。
テスト実行結果OK
画面上の動き
213d5a3 でassetの適用を行なっていますが、この部分まで課題として含めるとややボリュームが大きくsorcery自体の復習に集中できないような気がしています。
その場合は元々のmasterブランチにこちらのコミット内容をあらかじめ追加しておく対応を行おうかと考えています。