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와 DI 구현 #17

Merged
merged 23 commits into from
Nov 16, 2022
Merged

Coordinator와 DI 구현 #17

merged 23 commits into from
Nov 16, 2022

Conversation

jinwoong16
Copy link
Collaborator

📕 Issue Number

Close #16

📙 작업 내역

  • App
    • App DI Container를 추가하였습니다
    • App DI Container에서 지금은 각 DI Container를 생성해주는 메서드가 존재합니다. 추후에 service 관련 부분은 이곳을 통해서 각 DI Container로 보내주면 됩니다.
    • Coordinator를 추가하였습니다. start() 메서드를 통해 탭바 컨트롤러를 구성합니다. 현재는 홈, 둘러보기, 그리고 더보기가 존재합니다.
  • Home
    • Home DI Container를 추가하였습니다.
    • Coordinator를 추가하였습니다. start() 메서드를 통하여 HomeViewController를 실행합니다.
    • Home View Controller를 추가하였습니다. 지금은 배경색만 있습니다.
  • Browse
    • Browse DI Container를 추가하였습니다.
    • Coordinator를 추가하였습니다. start() 메서드를 통하여 BrowseViewController를 실행합니다.

📘 작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

📋 체크리스트

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • 변경사항이 효과적이거나 동작이 작동한다는 것을 보증하는 테스트를 추가하였는가?
  • 새로운 테스트와 기존의 테스트가 변경사항에 대해 만족하는가?

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 위에서 언급하였듯, 현재 App DI Container에는 다른 DI Container를 생성해주는 부분만 존재하는데, 추후에는 이곳에 DB Service관련 Injection을 추가하면 될 것 같습니다.

@jinwoong16
[feat] #10 - CustomProgressBar를 추가하였습니다.
0187223
@jinwoong16
[feat] close #10 - QuestCell을 재구성
807f0b4
@jinwoong16
close #10 - 주석을 추가와 이슈 종료
cc45533
@jinwoong16
[style] Quest Cell Constraints 수정
3a695b1
@jinwoong16
[style] QuestCell inset 변경
9b06b05
@jinwoong16
[feat] BrowseCell 정의
e5ca6bd 
수정할 사안
- 높이 동적으로 계산
- 헤더 동적으로 계산
@jinwoong16
[feat] 가짜 데이터를 생성하는 viewmodel 작성
41ca5a6
@jinwoong16
[feat] BrowseViewController 작성
6d332d1
@jinwoong16
[fix] Quest의 개수만큼 row의 개수가 처리되게 변경
895b3ff
@jinwoong16
[style] 가짜데이터 추가
437c10d
@jinwoong16
[fix] BrowseCell의 크기가 해당 위치의 퀘스트개수와 마진크기(20)를 포함하는 크기를 가지게 변경
240e9ac
@jinwoong16
[refactor] view did load 메서드 내부에 위치한 tableView를 설정하는 코드를 분리
22d7ece
@jinwoong16
[feat] header를 추가
30219bd 
- 이미지를 추가해야함.
@jinwoong16
[feat] DIContainer 구현
0816f9a
@jinwoong16
[feat] Coordinator 프로토콜 작성
d673792
@jinwoong16
[feat] AppCoordinator 작성.
2bef5c2 
- tabbar controller를 가지고 있습니다.
@jinwoong16
[feat] BrowseCoordinator 구현
aaf20d7
@jinwoong16
[feat] Home Coordinator 구현
bf2a63a
@jinwoong16
[feat] Home View Controller 추가
ed53e92 
- 현재는 ViewModel를 가지고 있지는 않습니다. 추후에 추가합시다 :)
@jinwoong16
[style] Scene Delegate 수정
2ec6465
@jinwoong16 jinwoong16 linked an issue Nov 16, 2022 that may be closed by this pull request
Coordinator와 DI #16
Closed
4 tasks
@jinwoong16
[feat] 아이콘 정보를 추가
cd97d76
@jinwoong16
[refactor] 탭바에 아이콘을 등록
9df88b9
@jinwoong16
[feat] App Appearance를 추가.
dedabd4 
- 탭바의 배경색이 흰색이고,
- 선택했을 때의 색상은 maxYellow여야 한다.
@jinwoong16 jinwoong16 merged commit 1a87f9f into develop Nov 16, 2022
@jinwoong16 jinwoong16 deleted the feature/Coordinators branch November 16, 2022 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feature 리뷰요청 진웅
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Coordinator와 DI
1 participant
@jinwoong16