Skip to content

OAuth 계정 refactoring #76

@minhyeok4dev

Description

@minhyeok4dev

기본적인 OAuth 로직은 짜여졌습니다. 다만 조금씩 refactoring 할 것들이 많이 보이고 있습니다.
다른 feature 개발하는 이슈들은 close하고, 해야할일에 대해서 앞으로 여기다 수정하며 계속 적어가면서 해결한 것들은 취소선 처리하겠습니다.

해당 이슈는 #64 Pull Request 와 연결됩니다.

  1. session에서 값을 추출하는 것보다 form의 hidden_field를 활용하도록 한다
  2. form의 URL은 {prefix}_path를 활용하도록 한다
  3. duplicated? 시 SQL쿼리를 날리지 않고 해결할 방법이 있을지 고민해보자
  4. Hash 사용 시 들여쓰기 수정한다
  5. self.from_omniauth 등 쓰이지 않는 코드는 삭제한다
  6. User.where(email: data["email"]).first대신 User.find_by_email(data["email"])를 활용한다
  7. 문서를 참고해 omniauth_callback_controller의 중복 제거해보기
  8. session을 거치지않고 수정해보기
  9. duplicate 저장말고 심볼을 변수와 함께 리턴해보기

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions