DonutWorks / Ari

0 stars 0 forks source link

#373 Feature/born activity #374

Closed MoojinChae closed 9 years ago

MoojinChae commented 9 years ago

373 Solved.

yhoonkim commented 9 years ago
angdev commented 9 years ago

acceptance/notice_spec 들이 깨져요 ㅠㅠ 아무래도 공지 화면에 활동 관련 기능이 덧붙여지다보니 element 관계가 달라져서 깨지는 문제인 것 같습니다 ㅠㅠ

minhyeok4dev commented 9 years ago

그리고 나도 이부분은 계속 고민인데 destroy dependent 를 걸어도 될까?? 왠지 이리저리 테스트해보다가 사용자가 다 날려버릴까봐..

MoojinChae commented 9 years ago

아 맞다! 테스트 수정 한다는거 깜빡했다 ㅠㅠ 의견 준것들 반영해서 일요일 밤에 커밋하고 머지 하겠습니다.

shaynekang commented 9 years ago

확인했습니다!

  1. 자주 이야기하지만, 컨트롤러에 뷰에 있는 whereorder는 모델로 빼 주시면 좋겠습니다.
  2. 그리고 사용하지 않는 테스트나 CSS/JS는 삭제해주세요. ㅎㅎ

이를 수정해주시면 될 것 같습니다. 수고하셨습니다~