DonutWorks / Ari

0 stars 0 forks source link

added validations to response model (presence of notice_id, user_id, uni... #396

Closed angdev closed 9 years ago

angdev commented 9 years ago

...queness of user_id on notice_id)

395

원래 없던 user_id, notice_id validation을 추가하고 user_id - notice_id 에 uniqueness validation을 추가하였습니다.

shaynekang commented 9 years ago

확인했습니다!

귀찮더라도 Model validation error는 flash보단 뷰에 직접 랜더링해주세요. response.errors.full_messages.join('<br />')같은 코드의 근본적인 원인은 에러를 계속 flash로 처리하기 때문입니다. 정말 특별한 상황이 아니라면 Model validation error에서 flash를 쓰지 않는 게 좋습니다.

그 외에는 문제 없는 것 같습니다. 수고하셨습니다~

angdev commented 9 years ago

토/일에 고치는걸로 (..)