search

DonutWorks / Ari

0 stars 0 forks source link

#362 move scheduler to rake task #372

Closed MoojinChae closed 9 years ago

MoojinChae commented 9 years ago

362 Solved.

shaynekang commented 9 years ago

헉, Rake Task를 빼는 코멘트는 Notice.deadline_send_sms의 내용을 rake로 빼면 모델 코드가 줄어드니까 좋을 것 같다는 이야기였는데...(먼산)

MoojinChae commented 9 years ago

아..! 그거에 맞게 수정 했습니다!

yhoonkim commented 9 years ago

확인 @MujinChae 앞으로 자동으로 문자를 많이 보내게 될 것 같은데, 현재 어떤 사람들한테 자동으로 알림이 가고 있는지 정리해놓은 테이블이나 문서가 있으면 좋겠어요.

angdev commented 9 years ago

확인했습니다. namespace가 db 인 이유는 있는지 궁금합니다!

@yhoonkim :+1:

minhyeok4dev commented 9 years ago

확인. 자동문자 생기면서 확실히 테스트할때 조심해야될 부분이 많아진것같긴함.

MoojinChae commented 9 years ago

일단 특별하게 뭐라 해야 할 지 몰라서 db로 했음 ㅋㅋ Rake Task 많이 생기면 연관 있는 거 묶어서 namespace 만듭시다! ㅎㅎ

MoojinChae commented 9 years ago

오토 머지가 안되는데.. 일요일 밤에 제가 머지 하겠습니다 ㅠㅠ

shaynekang commented 9 years ago

특별하게 뭐라 해야할지 모르겠다고 네이밍을 막 짓지 마세요. ㅋㅋ 협업에서 가장 중요한게 네이밍입니다. 내가 직접 설명해주지 않아도, 변수명&함수명만 보고서 어떤 역할을 하는지 파악할 수 있게 해야 합니다. ㅎㅎ

lib/tasks/deadline_check.rake에서 모델의 where를 많이 사용하는데, 이 부분만 모델로 빼면 될 것 같습니다.

여튼 수고하셨습니다~ -_-)/