Github 로그인 기능 구현 #72
Merged
Github 로그인 기능 구현 #72
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ASWebAuthenticationSession 사용
로그인 토큰의 작동 확인을 위해 get users 추가 구현
ellyheetov
reviewed
Jun 22, 2021
There was a problem hiding this comment.
다우니 혼자 정말 수고 많았어요
네트워크 코드 잘 짜신 것 같아요. 수정할 부분이 거의 없네요 ㅎㅎ
몇 가지 제가 궁금한 거나, 수정하는게 어떨까 하는 부분에 대해서 코멘트 달았어요.
그리고, 저희가 SwiftLint를 코코아팟으로 받아놓기는 했는데, 적용을 하지 않은 것 같아요.
여기 를 보면 쉽게 적용할 수 있을 것 같은데...
제가 적용해보니 Warning이 3,000개가 넘네요...?😅😅 이거 적용 해야하나요 말아야 하나요🤔
|
SwiftLint의 경우는 페어로 진행해보는 것도 좋을 것 같아요! |
dismiss()는 delegate 내부에 property로 전달.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
깃허브 로그인 기능 구현
애플 로그인의 경우, 백엔드의 기능 구현 우선순위에 따라서 후순위로 미뤄짐.
Network 모듈: 싱글톤. get 기능 구현
IssueListView에서 Network.shared의 로그인 토큰의 nil 여부를 확인하여 로그인 페이지 present.