-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Refactor] #196 - 디테일 성취 UI 로직 변경 #201
Conversation
guard let missionDailtData = self.missionDailyData else { return completion(nil) } | ||
completion(missionDailtData) | ||
} catch let err { | ||
print(err.localizedDescription, 500) |
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.
여기두 completion 있으면 좋을 것 같네여!
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.
고생하셨습니당 ㅎㅎ
HomeAPI.shared.getDailyMission(date: date) { response in | ||
guard let response = response else { return } | ||
guard let data = response.data else { return } | ||
self.updateData(item: data) |
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.
escaping 클로저 내부 self라서 weak self 함 해줘야 할듯합니다!
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.
수정했슴도ㅏㅇ~,~
🫧 작업한 내용
🔫 PR Point
기존 뷰는 VC -> background 뷰 -> title label과 collectionview로 구현하였습니다.
리팩토링하면서 background 뷰를 구현하지 않고 collectionview를 이용해서 기존의 title label을 collectionview의 header뷰로 구현했습니다.
📸 스크린샷
📮 관련 이슈