sessions#create 의 코드가 다소 지저분한 것 같네요. 리펙토링이 필요할 것 같습니다.
일단 지금 생각나는 건 크게 두 가지인데,
create_bookmark!가 컨트롤러가 아닌 다른 곳으로 가는 게 좋을 것 같아요.(모델 or 서비스 오브젝트)
- 저는
find_by_provider_and_uid와 create_with_omniauth를 find_or_create_with_omniauth 같은 이름으로 묶어서 구현하니 조금 더 깔끔해지더라구요. 마찬가지로 모델에 넣거나, 서비스 오브젝트에 넣으면 될 것 같습니다.
이정도네요. ㅎㅎ 파팅파팅!
sessions#create 의 코드가 다소 지저분한 것 같네요. 리펙토링이 필요할 것 같습니다.
일단 지금 생각나는 건 크게 두 가지인데,
create_bookmark!가 컨트롤러가 아닌 다른 곳으로 가는 게 좋을 것 같아요.(모델 or 서비스 오브젝트)find_by_provider_and_uid와create_with_omniauth를find_or_create_with_omniauth같은 이름으로 묶어서 구현하니 조금 더 깔끔해지더라구요. 마찬가지로 모델에 넣거나, 서비스 오브젝트에 넣으면 될 것 같습니다.이정도네요. ㅎㅎ 파팅파팅!