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.

Sign up for GitHub

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

Jump to bottom

bibi의 리팩토링 - 코드리뷰 팔로우업 #28

Closed
5 tasks done
bibi6666667 opened this issue Jul 19, 2022 · 0 comments
Closed
5 tasks done

bibi의 리팩토링 - 코드리뷰 팔로우업 #28

bibi6666667 opened this issue Jul 19, 2022 · 0 comments
Assignees
@bibi6666667

Comments

@bibi6666667
Copy link
Collaborator

bibi6666667 commented Jul 19, 2022 •
edited
Loading

캉운이 리뷰해주시며 추천한 내용대로 리팩토링 해보기!

  • Environment 적용 - AppDelegate
  • Environment 적용 - ReposModel
  • Issue, NewIssue, OptionSelect에도 Environment 적용하기
  • Coordinator 패턴 공부하기
  • Container에서 Coordinator 역할 분리하기
@bibi6666667 bibi6666667 self-assigned this Jul 19, 2022
bibi6666667 added a commit that referenced this issue Jul 19, 2022
@bibi6666667
[Jinsujin/issue-tracker] #28 Container의 의존성을 관리하는 ContainerEnvironmen…
94f6cb2 
…t 추가
bibi6666667 added a commit that referenced this issue Jul 20, 2022
@bibi6666667
[Jinsujin/issue-tracker #28] ReposModel의 의존성인 IssueService.requestRep…
2cf9ed0 
…os()를 위해 service 전체가 아닌 클로저만 Environment로 받도록 변경
bibi6666667 added a commit that referenced this issue Jul 20, 2022
@bibi6666667
Revert "[Jinsujin/issue-tracker #28] ReposModel의 의존성인 IssueService.re…
c5bca22 
…questRepos()를 위해 service 전체가 아닌 클로저만 Environment로 받도록 변경"

This reverts commit 2cf9ed0.
bibi6666667 added a commit that referenced this issue Jul 20, 2022
@bibi6666667
[Jinsujin/issue-tracker #28] ReposModel의 의존성인 IssueService.requestRep…
f1dc0ce 
…os()를 위해 service 전체가 아닌 클로저만 Environment로 받도록 변경
bibi6666667 added a commit that referenced this issue Jul 21, 2022
@bibi6666667
[Jinsujin/issue-tracker] #28 IssueModel이 service 전체를 받지 않고, service에서…
d791820 
… 사용하는 클로저만 environment로 받도록 변경
bibi6666667 added a commit that referenced this issue Jul 21, 2022
@bibi6666667
[Jinsujin/issue-tracker] #28 Feat: NewIssueModel에 service 전체 대신, 필요한 …
aaf1c2f 
…클로저만 environment로 주입
bibi6666667 added a commit that referenced this issue Jul 21, 2022
@bibi6666667
[Jinsujin/issue-tracker] #28 Feat: OptionSelect에서 사용하는 service의 메서드들을…
7e9b7ba 
… 클로저로 주입하도록 변경
bibi6666667 added a commit that referenced this issue Jul 25, 2022
@bibi6666667
[Jinsujin/issue-tracker] #28 Refactor: LoginVC와 LoginModel 분리 및 OAuth…
3dce5a4 
…Service에서 필요한 클로저만 갖도록 환경 분리
bibi6666667 added a commit that referenced this issue Jul 25, 2022
@bibi6666667
[Jinsujin/issue-tracker] #28 Refactor: IssueVC 생성 시 필요한 작업을 viewDidLo…
2aa67de 
…ad대신 Container에서 하도록 변경
bibi6666667 added a commit that referenced this issue Jul 25, 2022
@bibi6666667
[Jinsujin/issue-tracker] #28 Refactor: OptionSelectVC 생성 시 필요한 설정 부분을…
da4fa97 
… viewDidLoad 대신 Container에서 처리하도록 변경
bibi6666667 added a commit that referenced this issue Sep 2, 2022
@bibi6666667
Feat: IssueViewController 화면 흐름을 관리하는 IssueCoordinator 추가 #28
1065a7b
bibi6666667 added a commit that referenced this issue Sep 2, 2022
@bibi6666667
Feat: 모든 뷰컨에 coordinator 프로퍼티 추가 및 NewIssue 생성 시 뒤로가기 동작 구현 #28
300373e
bibi6666667 added a commit that referenced this issue Sep 2, 2022
@bibi6666667
Feat: OptionSelectViewController를 관리하는 OptionSelectCoordinator 추가 #28
7170c83
bibi6666667 added a commit that referenced this issue Sep 2, 2022
@bibi6666667
Feat : coordinator생성 시 register한 뒤, childCoordinators 관리할 때 활용 #28
9f80da2 
기존 VC에서 coordinator를 직접 소유하던 부분을 삭제하고
AppCoordinator에서 childCoordinators를 관리할 때 register된 coordinator 사용
bibi6666667 added a commit that referenced this issue Sep 2, 2022
@bibi6666667
Feat : 이슈생성, 옵션선택 후 coordinator에서 이전 화면으로 되돌아가며 reload호출 (동작하지 않음) #28
e3a03eb
bibi6666667 added a commit that referenced this issue Sep 2, 2022
@bibi6666667
Feat : OptionSelectVC를 coordinator로 연결 - 옵션마다 다른 화면 생성 #28
c94cb85
bibi6666667 added a commit that referenced this issue Sep 5, 2022
@bibi6666667
Feat : 이슈 생성 요청과 이슈목록 요청 사이에 딜레이 추가, 받아온 이슈에 새로 생성한 이슈가 없다면 재요청 #28
7e2c599
bibi6666667 added a commit that referenced this issue Sep 5, 2022
@bibi6666667
Chore : 미사용 메서드와 주석 삭제 #28
cf7c960
bibi6666667 added a commit that referenced this issue Sep 6, 2022
@bibi6666667
Chore : 불필요한 주석 및 코드 삭제 #28
00867fb
bibi6666667 added a commit that referenced this issue Sep 22, 2022
@bibi6666667
refactor: ViewController deinit이 호출되지 않는 문제 디버깅 #28
5865265 
- delegate 프로토콜을 weak var로 갖도록 변경
- 클로저 내에 선언된 self 를 weak self로 변경
bibi6666667 added a commit that referenced this issue Sep 22, 2022
@bibi6666667
refactor: 변수명을 적절하게 수정 #28
75a66ca
bibi6666667 added a commit that referenced this issue Sep 22, 2022
@bibi6666667
refactor: 불필요한 self 키워드 삭제 #28
264ed97
bibi6666667 added a commit that referenced this issue Sep 22, 2022
@bibi6666667
refactor: 이슈 생성 요청 이후 새 이슈가 생성되었는지 확인한 다음 이슈 목록으로 되돌아가도록 변경 (뷰 미완성) #28
d37d8c9
bibi6666667 added a commit that referenced this issue Sep 22, 2022
@bibi6666667
refactor : 새 이슈 생성 -> 이슈 조회 흐름 수정. #28
55b3b87 
- 새 이슈 생성 시, "이슈생성 요청" 이후 "이슈목록 조회 요청" 을 보내 생성된 이슈가 반영될때까지 시간 간격을 두고 재요청을 한다. 이 때 indicator를 나타낸다.
- 반영이 되면 이전 화면인 이슈목록으로 돌아가 이슈목록을 조회하고, 화면을 갱신한다.
bibi6666667 added a commit that referenced this issue Sep 22, 2022
@bibi6666667
debug : 로그인 직후 앱으로 돌아왔을 때 저장소 목록이 나오지 않는 현상 디버깅(해결) #28
0d2db84
bibi6666667 added a commit that referenced this issue Sep 22, 2022
@bibi6666667
refactor : Container에서 뷰컨트롤러와 관련 코디네이터를 묶어 관리하고 삭제 작업도 처리하도록 변경 #28
6e2fcec
bibi6666667 added a commit that referenced this issue Sep 22, 2022
@bibi6666667
refactor : 뷰컨트롤러와 관련 코디네이터 이름을 담은 ViewControllerCoordinator 열거형을 사용해 …
448b9dd 
…중복 코드 개선 #28
bibi6666667 added a commit that referenced this issue Sep 22, 2022
@bibi6666667
chore : 불필요한 주석 및 print문 삭제 #28
f44007b
bibi6666667 added a commit that referenced this issue Sep 26, 2022
@bibi6666667
debug : IssueService 초기화를 위해 토큰 기본값을 넘기는 대신 옵셔널로 타입 수정 #28
c5d1b0b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bibi6666667 bibi6666667

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bibi6666667