[Feat] #388 - 솝마디 메인 뷰 API 연결 #390
Conversation
- ViewControllable -> Presentable로 변경
|
| @@ -9,6 +10,7 @@ | |||
| import Combine | |||
|
|
|||
| public protocol DailySoptuneRepositoyInterface { | |||
There was a problem hiding this comment.
제가 오타 냈었나바요.. DailySoptuneRepositoyInterface -> DailySoptuneRepositoryInterface로 수정 오네가이시마스 ....🥹
| extension DailySoptuneCardVC { | ||
| private func setUI() { | ||
| private extension DailySoptuneCardVC { | ||
| func setUI() { |
There was a problem hiding this comment.
가독성을 위해 바꾸신 건가요?.? 궁금해요
There was a problem hiding this comment.
#387 에서 리뷰 받은건데, extension 앞에 private을 붙이면 extension 내부에 있는 모든 메소드와 프로퍼티가 private 지정이 된다 하더라구요 ! 어차피 extension 내부는 전부 private일 것 같아서 같이 수정했습니다 !
| @@ -98,4 +123,15 @@ extension DailySoptuneMainVC { | |||
| make.height.equalTo(270.adjustedH) | |||
| } | |||
| } | |||
|
|
|||
| func bindViewModel() { | |||
There was a problem hiding this comment.
요기.. 앞으루 bindViewModel -> bindViewModels로 통일할까요?! 🫶🏻
| @@ -10,14 +10,20 @@ import Foundation | |||
| import Combine | |||
|
|
|||
| import Moya | |||
| import Network | |||
There was a problem hiding this comment.
혹시 import Network하신 이유가 있나요?.?
There was a problem hiding this comment.
어라 충돌 해결하면서 제가 잘못 옮긴 것 같아요 ! 지우겠습니다
|
아! 그리고 저도 DailySoptuneResultVC의 dateLabel에 쓰이는 setCurrentDateString()을 VM에 구현해뒀는데, String+으로 빼두 될 것 같아요!.! 수정해주시면 같이 반영하겠습니다!! 🙏 |
🌴 PR 요약
🌱 작업한 브랜치
🌱 PR Point
📌 참고 사항
📮 관련 이슈