Masaki Komagata
Masaki Komagata
@AyakaTakashima > ただ、daiki0381さんが作成したものを私が作成したというのは間違えで、このPRで私が作成しようと思っているものをER図に起こしたと認識していただけますと幸いです。 なるほどです。認識しました。
@AyakaTakashima surveryが複数のsurvey_questionをもつのだとしたら、surveryのsurvey_question_idだと複数のものを表現できないかもです〜。
@AyakaTakashima これだとsurvey_questionは一つのsurveyにしか所属できないように思います〜
@AyakaTakashima 中間テーブルになった部分はOKです〜。 ただ、`reply_deadline_at`は`created_at`や`published_at`のように動詞じゃないと`_at`が英語的に成り立たないので、名前を検討してみていただきたいです〜
@AyakaTakashima 冗長な表現は避けたいので直訳を避けて`expires_at`などはいかがでしょうか。
@pachikuriii > これはイベント編集ページのフォームと同様の仕様になっています。(ただ、イベント編集ページにおいてはフォームのバリデーションが働いて、秒数が実質変更できない状態です。) 今ままで大丈夫です〜。 イベントの秒数が変更できないのはバグなので、もし可能であれば、バグとしてIssue登録お願いできればありがたいです〜!
@pachikuriii なるほどです。 ただ、日時カラムなので、記事公開日もイベントの日時も秒数まで変更できる仕様でお願いします〜
@fuwa-syugyo (CC: @machida ) preventDefaultはデフォルトの動作をやらないようにするものですが、それを削除したことが影響してたりしないでしょうか?
@machida これ良いですね!
@yamatatsu10969 https://github.com/fjordllc/bootcamp/issues/5560#issuecomment-1275479350 こちらはメンターへの質問という意味でしょうか? もしそうだとしたら、メンションがないと気づくことができないので今後はメンションをお願いできれば助かります〜。 (他の方のPRやIssueも見ていただけるとどのようなコミュニケーションをとっているのか参考になるかもです〜) 独り言であれば、 :memo: xxxxx みたいな書き方をした方が、見た人がわかりやすいかもです。 > fixtures に追加した user などは、 rails seed コマンドでDBに追加されるのか `rails db:seed`で追加されます。しかし、fixturesは二箇所あります。 `db/fixtures`はdevelopment環境のデータで、`test/fixtures`はtest環境のデータです。 > collegue(正しくはcolleague) をタイポしているので修正したいのだが別チケットにするべきか 見つけてくださってありがとうございます。このぐらいであれば、コミットが別であれば同一Issue内で治してしまって大丈夫です〜。