DonutWorks / Ari

0 stars 0 forks source link

#320 관리자는 사용자에게 tag를 달 수 있고, 태그별로 sms를 전송할 수 있다. #351

Closed yhoonkim closed 9 years ago

yhoonkim commented 9 years ago

320 issue 입니다.

후아...

minhyeok4dev commented 9 years ago
yhoonkim commented 9 years ago

민혁

shaynekang commented 9 years ago
  1. user_user_tag 대략 조치 안타. ㅋㅋ 더 좋은 네이밍 없을까요?
  2. admin/users#createadmin/users#update 에서 begin - rescue로 모든 에러를 다 잡아버리는데, 이렇게 하면 구체적으로 어떤 에러가 났는지 알 수 없습니다. 방금도 테스팅겸 하나 고쳤다가 에러가 났는데 아무 반응이 없어서 고생했네요. ㅎㄷ 에러가 나면 명시적으로 어떤 에러가 나는지 알려줘야 합니다.
  3. UserTag.find_by_tag_name(term) || UserTag.create(tag_name: term)UserTag.find_or_create_by(tag_name: term) 로 한 방에 해결될 것 같습니다. ㅎㅎ
  4. pretty_tags는 또 뭐여 ㅋㅋㅋ 그리고 이런건 데코레이터로 빼도록 합시다. ㅎㅎ
  5. models/user_tag.rb에서 SQL LIKE보다는 Arel을 사용하는 걸 권장합니다. 가끔씩 Native SQL쿼리가 한 DB에서는 돌아가는데 다른 DB에서는 안 돌아가는 경우가 있어요. Sqlite3에서는 잘 돌아가는데 PostgresSQL에서는 안 돌아가는거죠.
  6. 그리고 사용하지 않는 테스트는 지우면 좋을 듯 합니다. ㅎㅎ

나머지는 조금 더 확인해보고 다시 올리겠습니다~