cyder / Aktiva-V2-Server

API server + docker
0 stars 0 forks source link

歌詞のtextが空文字列でも許可する #52

Closed mori-atsushi closed 7 years ago

mori-atsushi commented 7 years ago

close #51

euglena1215 commented 7 years ago

@Mori-Atsushi Rails側で実装するのもいいんですが、 DBのテーブル側にdefault: ""を追加して バリデーションを外してやればいいだけな気がするんだけどいかがでしょう? (テスト書かなくてよくなるし)

mori-atsushi commented 7 years ago

それだとtextを指定しなかった時に勝手に空文字として入っちゃうよね? ミス防止のためにも、textを明示的に空文字として記述した時以外はエラーを出したい。

euglena1215 commented 7 years ago

把握しましたー

euglena1215 commented 7 years ago

LGTM