-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Feat] 서비스 타입 구현 #192
[Feat] 서비스 타입 구현 #192
Conversation
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.
고생하셨습니다.
let service: InviteCodeServiceType | ||
let inviteCode = ObservablePattern<String>("") | ||
let inviteCodeState = ObservablePattern<InviteCodeState>(.empty) | ||
let isNextButtonEnabled = ObservablePattern<Bool>(false) | ||
|
||
private let service: InviteCodeServiceType | ||
|
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.
service를 ViewModel 외부에서 접근가능하게 선언하신 이유가 궁금하네요?
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.
사실 막연하게 쓸 일이 있을 것 같아서 일단 바꿔놨는데 생각해보니 필요가 없네요 ,,, 감사합니다!
|
||
let tardyService: MockTardyService | ||
var isPastDue: ObservablePattern<Bool> |
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.
tardyServiceType을 주입받으면 어땠을까 하는..
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.
수정했습니다 ~!
protocol PromiseInfoServiceType { | ||
func getPromiseInfo(with promiseId: Int) -> PromiseInfoModel? | ||
} | ||
|
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.
저는 get보다는 fetch를 더 많이 사용합니다.
그냥 그렇다구요
Conflicts: KkuMulKum.xcodeproj/project.pbxproj
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.
이제 의존성 분리와 MVVM 그리고 서비스 객체 나누는것까지 완전하게 이해하신것같네요! 응원합니다!
|
||
// MARK: Initialize | ||
|
||
init(service: CreateMeetingServiceType) { | ||
self.service = service | ||
init(createMeetingService: CreateMeetingServiceType) { |
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.
아주 좋아용 수고하셨습니다~!
🔗 연결된 이슈
📄 작업 내용