search

DonutWorks / Ari

0 stars 0 forks source link

#384 Feature/to notice can be ended by user #387

Closed MoojinChae closed 9 years ago

MoojinChae commented 9 years ago

384 solved.

shaynekang commented 9 years ago
  1. 자주 이야기하지만(정말 자주 이야기하는 것 같네요 -_-;;) validation error가 나면 명확한 사유를 보여줘야 합니다. 정말 특별한 상황이 아니면 Model validation error는 flash로 처리하지 않고, error를 그대로 보여주세요. admin/notices#to_notice_end_deadline 처럼 "공지를 마감 할 수 없습니다." 라고 보여주고 그냥 넘어가면 안됩니다.
  2. 마찬가지로 자주 이야기하지만, 뷰에서 만드는 로직 코드는 다른 곳으로 빼는게 좋습니다. Time.now.localtime.strftime("%Y-%m-%d %T") 같은 경우는 저라면 헬퍼로 빼거나, @notice.due_date > Time.now.localtime.strftime("%Y-%m-%d %T")를 통채로 @notice.deadline?으로 뺄 것 같네요.

이상입니다. 수고하셨습니다~

MoojinChae commented 9 years ago

수정 하고 머지 했습니다! ㅎㅎ

shaynekang commented 9 years ago

음 이번에는 이런 에러가 나는군요.

22_no_admin_to_response_index

views/admin/to_responses/index.html.erb가 없다고 나오는 것 같습니다. ㅎㅎ