-
Notifications
You must be signed in to change notification settings - Fork 1
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 [#360] 3차 스프린트 건의사항 반영 #361
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.
P5
고생하셨습니다!!
let startOfToday = calendar.startOfDay(for: currentDate) | ||
let startOfTappedDate = calendar.startOfDay(for: date) | ||
|
||
return calendar.dateComponents([.day], from: startOfTappedDate, to: startOfToday).day ?? 0 > 0 |
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.
여기서 혹시 0>0은 bool 값으로 보내주는 건가요?
혹시 그냥 false로 설정하지 않은 이유가 있을까요?
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.
(day ?? 0 ) > 0 으로 봐주시면 될 것 같습니다 !!
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.
P5. 고생했습니다! 최고최고
@@ -96,13 +96,19 @@ extension NotificationView { | |||
isTapped.toggle() | |||
doNotSeeAgainButton.setImage(UIImage(imageLiteralResourceName: isTapped ? "btnCheckBox" : "btnNotCheckBox"), for: .normal) | |||
UserDefaults.standard.set(isTapped, forKey: "isTapped") | |||
if isTapped { | |||
UserDefaults.standard.set(Date(), forKey: "whenTapped") |
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.
P5. 진짜진짜 의견인데 whenTapped
보다는 TapDate
와 같이 명사 형태가 좋을 것 같습니다! isTapped
는 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.
헉 넵 반영하겠습니다 !!
⛏ 작업 내용
📌 PR Point!
📸 스크린샷
✅ Issue
Resolved #360