Repository navigation
[fix] #760 일기 날짜 선택 제목을 '기록 날짜' 로 바꾼다 - #899
Merged
Merged
Conversation
이 날짜는 과거 일기를 몰아 쓰기 위한 기록 날짜인데 제목이 '발송 날짜' 였다. 시안이 2026-08-05 에 '기록 날짜' 로 수정돼 그에 맞춘다. 이슈 노트의 제안대로 리소스 키의 `send_date` 명칭도 `record_date` 로 함께 정리한다 — 값만 바꾸면 다음 사람이 키를 보고 발송용으로 오해한다. `ReceiverMindRecordFilterSheet` 의 '발송 날짜' 는 수신자 열람 관점 문구라 손대지 않는다. Closes #760 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
OnlyReadOneBook
approved these changes
Aug 23, 2026
OnlyReadOneBook
left a comment
Collaborator
There was a problem hiding this comment.
승인합니다. 지적 없습니다.
검증:
- CI 전 체크 green (스크린샷 검증 포함)
- 부작용 전수 대조: mindrecord 래퍼
component/BottomSheetCalendar.kt의 호출부는 develop 전체에서DiaryWriteScreen.kt:273단 한 곳 — setting·timeletter가 쓰는 것은 title을 직접 전달받는 core:ui의 동명 컴포넌트라 무영향, 데일리질문 작성에는 캘린더 없음 - strings.xml 리소스 유지(하드코딩 아님), 키까지
record_date로 정리, 옛 키send_date_title잔존 참조 0건 - 이슈 노트대로 수신자 관점 '발송 날짜'(
ReceiverMindRecordFilterSheet.kt:81)는 비변경 확인
Collaborator
|
시안 대조 보강 — 일기 날짜 선택 바텀시트 시안(2671:17879)의 시트 제목이 "기록 날짜"임을 직접 확인했습니다. 변경 문구가 시안과 일치합니다. |
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌𝘐𝘴𝘴𝘶𝘦𝘴
Closes #760
📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯
시안 수정(2026-08-05, 류하늬 님 회신)에 맞춰 문구를 바꿨습니다.
이슈 노트의 제안대로 리소스 키도 함께 정리했습니다.
값만 바꾸고 키를
send_date로 두면 다음 사람이 발송용 문구로 오해합니다. 참조는BottomSheetCalendar.kt:18한 곳뿐이라 개명 비용이 없었습니다. 왜 '발송' 이 아닌지도 리소스 주석에 남겼습니다.📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵 · 실측
에뮬레이터 debug 빌드 → 기록 탭 → 일기 → 작성 → 날짜 탭:
💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴
:feature:mindrecord:presentation:compileDebugKotlin+ktlintCheck+:app:assembleDebugBUILD SUCCESSFUL, 위 실측.ReceiverMindRecordFilterSheet.kt:81의 '발송 날짜' 는 수신자 열람 관점 문구라 건드리지 않았습니다.