-
Notifications
You must be signed in to change notification settings - Fork 0
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
SOPT 세미나 4차 과제 #8
base: develop
Are you sure you want to change the base?
Conversation
- Info.plist 세팅 및 xcconfig 파일 추가 - NetworkTestController 추가
- ServerHomeController 내 유저 등록 기능까지 구현 - NetworkError, UserService 등 네트워킹 관련 클래스 추가
- ProfileCell, PlainCell 추가 - ServerHomeController 초기 구현
- 로그인 실패 시 로그인 실패 팝업 노출되도록 구현 - 로그인 페이지 UI 개선
- plainCell의 경우, PlainCellTag로 구분할 수 있도록 수정 - plainCell, plainCellWithRightArrow 통합 - profileCell, ProfilePlaceholderCell 구분
- logout 섹션을 별도로 구현하여, 맨 아래에 로그아웃 셀 위치하도록 수정
- 회원가입 페이지 내부 UI 레이아웃 수정
- MyHobbyViewController 제거 후, ServerHomeController에서 사용자 취미 plain 셀로 노출하도록 변경 - PlainCell 내부 코드 수정 - showAlert extension 추가
- UserInfEditController 추가 - UserService 클래스 내 코드 정리
- DTO 파일 분리 - Foundation / Feature / Infrastructure로 폴더 구조 정리 - UserAuthenticator, UserAuthCredential, TokenManager 구현 - token 담아서 보내는 로직 interceptor에서 처리하도록 변경
- 모달 클릭 시 키보드 내려가도록 구현
case .failure: | ||
let networkError = self.handleStatusCode(statusCode, data: Data()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
실패 시 Data() 라는 빈 데이터를 넘기는 방법을 사용하셨는데,
response.data를 옵셔널 바인딩하는 방법은 어떻게 생각하시는지 궁금합니다 ..!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리팩토링 해보겠습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!!
- Environment 위치 이동
1️⃣ What is this PR?
2️⃣ Description
1) 4차 과제 진행
3️⃣ To Reviewers
1. 토큰 활용
(token refresh api가 따로 존재하지는 않아서, 토큰 재발급 기능은 구현하지 않았습니다.)
2. 기타
4️⃣ ScreenShots
week4_assignment.mp4