We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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 할 것들이 많이 보이고 있습니다. 다른 feature 개발하는 이슈들은 close하고, 해야할일에 대해서 앞으로 여기다 수정하며 계속 적어가면서 해결한 것들은 취소선 처리하겠습니다.
해당 이슈는 #64 Pull Request 와 연결됩니다.
self.from_omniauth
User.where(email: data["email"]).first
User.find_by_email(data["email"])
The text was updated successfully, but these errors were encountered:
#76 리팩토링
68e7e63
페이스북과 구글이 중복되는 로직을 띄고있어 내심 좀 불편했는데요. 찾다가 이를 하나로 묶어볼 수 있을것같은 문서를 하나 발견해서 중복 제거도 한번 시도해보겠습니다.
Sorry, something went wrong.
dat class_eval..
#76 7~9번 리팩토링
ab02179
#76 다양한 예외사항 처리
9166136
minhyeok4dev
No branches or pull requests
기본적인 OAuth 로직은 짜여졌습니다. 다만 조금씩 refactoring 할 것들이 많이 보이고 있습니다.
다른 feature 개발하는 이슈들은 close하고, 해야할일에 대해서 앞으로 여기다 수정하며 계속 적어가면서 해결한 것들은 취소선 처리하겠습니다.
해당 이슈는 #64 Pull Request 와 연결됩니다.
session에서 값을 추출하는 것보다 form의 hidden_field를 활용하도록 한다form의 URL은 {prefix}_path를 활용하도록 한다duplicated? 시 SQL쿼리를 날리지 않고 해결할 방법이 있을지 고민해보자Hash 사용 시 들여쓰기 수정한다self.from_omniauth
등 쓰이지 않는 코드는 삭제한다User.where(email: data["email"]).first
대신User.find_by_email(data["email"])
를 활용한다문서를 참고해 omniauth_callback_controller의 중복 제거해보기session을 거치지않고 수정해보기duplicate 저장말고 심볼을 변수와 함께 리턴해보기The text was updated successfully, but these errors were encountered: