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

Coordinator Pattern 및 부분적인 ReactorKit 적용, 기타 코드 리팩토링 작업을 진행합니다. #38

Closed
sanghyeok-kim opened this issue Jul 28, 2023 · 0 comments · Fixed by #39
Closed

Coordinator Pattern 및 부분적인 ReactorKit 적용, 기타 코드 리팩토링 작업을 진행합니다. #38

sanghyeok-kim opened this issue Jul 28, 2023 · 0 comments · Fixed by #39
Assignees
@sanghyeok-kim
Labels
refactor 로직에 대한 변경 없는 코드 변경
Milestone
타이머 추가, 설정 화면 구현

Comments

@sanghyeok-kim
Copy link
Owner

sanghyeok-kim commented Jul 28, 2023 •
edited
Loading

  • 화면 전환 로직에 대해 Coordinator Pattern 적용
  • TimerSettingViewModel, TimerSettingViewController에 대해 ReactorKit을 적용하여 각각 View, Reactor 타입으로 리팩토링
  • TimerCreateViewModel, TimerCreateViewController에 대해 ReactorKit을 적용하여 각각 View, Reactor 타입으로 리팩토링
  • 'TimerSetting -> TimerEditing', 'Main -> Home'으로 각각 네이밍 수정
  • 커스텀 뷰의 이벤트 처리에 Relay 프로퍼티를 별도로 사용하던 방식에서 RxCocoa Traits를 사용하도록 변경
@sanghyeok-kim sanghyeok-kim self-assigned this Jul 28, 2023
@sanghyeok-kim sanghyeok-kim added the refactor 로직에 대한 변경 없는 코드 변경 label Jul 28, 2023
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 feat: Podfile에 ReactorKit 의존성 추가
34659cd
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 feat: Coordinator 프로토콜, CoordinatorType 열거형 생성 및 구현
64b1e67
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 feat: HomeCoordinator 타입 및 HomeCoordinatorAction 열거형 구현
c784ed0
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 refactor: SceneDelegate에서 HomeCoordinator를 통해 화면 플로우를 시작하도록 변경
4233d68
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 refactor: TagButton와 TagScrollView에 대한 Reactive Extension 구현, 이벤트…
9b9b7cb 
… 처리를 ControlEnvent와 Binder로 처리할 수 있도록 변경
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 refactor: UITextField의 textChanged ControlProperty를 ControlEvent로 변경
33f49c5 
- 네이밍과 역할을 봤을 때 ControlProperty(getter와 setter를 모두 가짐)보다는 ControlEvent(이벤트를 방출만 함)가 더 적절함
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 remove: NameTextField의 불필요한 코드 제거
e5fc627 
- TextField의 text에 접근하는 별도의 프로퍼티를 만들 필요 없이 rx.text를 직접 사용하면 되므로 불필요, 따라서 제거함.
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 refactor: MainViewController, ViewModel에 있던 화면 전환 책임을 Coordinator…
64741cd 
…가 수행하도록 변경
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 refactor: MainViewModel의 중복되는 코드를 함수로 분리
09921dd
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 refactor: TimerCellViewModel에 있던 화면 전환 책임을 Coordinator가 수행하도록 변경
cf79729
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 refactor: TimerCreateViewController, TimerCreateViewModel에 대해 Rea…
f0d0b60 
…ctorKit을 이용하여 각각 View, Reactor 타입으로 리팩토링
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 refactor: TimerSettingViewController, TimerSettingViewModel에 대해 R…
2b9dcab 
…eactorKit을 이용하여 각각 View, Reactor 타입으로 리팩토링
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 chore: settingTimer에 대한 English 로컬라이징 텍스트 수정 (Setting -> Editing)
163ed7c
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 rename: 네이밍 수정 (TimerSetting -> TimerEditing)
9e00f15
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 rename: 네이밍 수정 (Main -> Home)
8033785
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
#38 remove: 불필요한 코드 제거
687c928
@sanghyeok-kim sanghyeok-kim changed the title 부분적으로 ReactorKit을 적용하고 코드 리팩토링을 진행합니다. Coordinator Pattern 및 ReactorKit 적용, 기타 코드 리팩토링 작업을 진행합니다. Jul 29, 2023
@sanghyeok-kim sanghyeok-kim changed the title Coordinator Pattern 및 ReactorKit 적용, 기타 코드 리팩토링 작업을 진행합니다. Coordinator Pattern 및 부분적인 ReactorKit 적용, 기타 코드 리팩토링 작업을 진행합니다. Jul 29, 2023
@sanghyeok-kim sanghyeok-kim linked a pull request Jul 29, 2023 that will close this issue
Coordinator Pattern 및 부분적인 ReactorKit 적용, 기타 코드 리팩토링 작업 #39
Merged
5 tasks
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
Merge branch 'main' into feature/#38-ReactorKit-Refactoring
3ba57e3
sanghyeok-kim added a commit that referenced this issue Jul 29, 2023
@sanghyeok-kim
[Merge pull request] #39 from sanghyeok-kim/feature/#38-ReactorKit-Re…
3a18675 
…factoring

Coordinator Pattern 및 부분적인 ReactorKit 적용, 기타 코드 리팩토링 작업
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sanghyeok-kim sanghyeok-kim

Labels
refactor 로직에 대한 변경 없는 코드 변경
Projects
None yet
Milestone
타이머 추가, 설정 화면 구현
1 participant
@sanghyeok-kim