search

DonutWorks / Ari

0 stars 0 forks source link

removed SendSMS #349

Closed angdev closed 9 years ago

angdev commented 9 years ago

340

renamed SendSMS2 to SMSSender. Original SMSSender is Admin::Messages::SendMessageService now.

(applied #338, #339)

shaynekang commented 9 years ago

확인했습니다~!

음... services를 Admin 네임스페이스로 감싸줄 필요가 있을지는 모르겠네요. ㅎㅎ 저 같은 경우는 서비스 오브젝트가 많아지면 폴더를 나눠주긴 하는데,

  1. 컨트롤러보다 모델 기준으로 나눠주고.
  2. 네임스페이스를 쓰지 않고 그냥 모델명을 서비스 앞에 붙입니다.

가령 Admin::Messages::SendMessageService 같은 경우는 MessageSenderService 로 네이밍을 하고, services/message/message_sender_service.rb에 넣어주죠. 이건 개인 성향의 차이라, 참고하시고 지금이 더 좋다고 판단하면 패스하셔도 됩니다. ㅎㅎ