Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OAuth 계정 refactoring #76

Open
minhyeok4dev opened this issue Aug 27, 2014 · 2 comments
Open

OAuth 계정 refactoring #76

minhyeok4dev opened this issue Aug 27, 2014 · 2 comments
Assignees

Comments

@minhyeok4dev
Copy link

기본적인 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 저장말고 심볼을 변수와 함께 리턴해보기
@minhyeok4dev
Copy link
Author

페이스북과 구글이 중복되는 로직을 띄고있어 내심 좀 불편했는데요. 찾다가 이를 하나로 묶어볼 수 있을것같은 문서를 하나 발견해서 중복 제거도 한번 시도해보겠습니다.

@angdev
Copy link
Member

angdev commented Aug 27, 2014

dat class_eval..

minhyeok4dev pushed a commit that referenced this issue Aug 27, 2014
minhyeok4dev pushed a commit that referenced this issue Aug 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants