Merged
Conversation
fix: 습득 날짜 수정
fix: status가 null이면 전체 조회
fix: 소유권 신청 후 승인이 되면 나머지 학생들의 소유권 거절
- 습득일(foundAt) 대신 등록일(createdAt)을 기준으로 폐기 예정일 계산 - markAsToBeDiscarded() 메서드: discardedAt = createdAt + 6개월 Co-Authored-By: jaeminjo732 <[email protected]>
- findByStatusAndFoundAtBefore → findByStatusAndCreatedAtBefore - 등록일 기준 조회를 위한 메서드명 갱신 Co-Authored-By: jaeminjo732 <[email protected]>
- ItemFacade: 메서드명 변경 (findByStatusAndFoundAtBefore → findByStatusAndCreatedAtBefore) - ItemSchedulerService: 등록일 기준 조회로 변경 - 로그 메시지: "습득일" → "등록일"로 수정 Co-Authored-By: jaeminjo732 <[email protected]>
- 폐기 예정일 기준 변경 반영 (습득일 → 등록일) - 워크플로우, 로그 예시, 테스트 시나리오 업데이트 - 버전 1.3으로 갱신 및 변경 이력 추가 Co-Authored-By: jaeminjo732 <[email protected]>
- DatePrecisionParserTest 추가 (28개 테스트 케이스) - 정상 파싱: yyyy, yyyy-MM, yyyy-MM-dd 형식 - 예외 처리: null, 빈 문자열, 잘못된 형식, 유효하지 않은 날짜 - DatePrecisionParser에 ResolverStyle.STRICT 적용 - 유효하지 않은 날짜 자동 조정 방지 (2025-02-30 → 예외 발생) - yyyy를 uuuu 패턴으로 변경하여 STRICT 모드 호환성 확보 Co-Authored-By: jaeminjo732 <[email protected]>
Refactor/181 disposal base change
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
3 participants
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.
📋 요약
이 PR에서 수행한 작업에 대한 간단한 설명을 작성해주세요.
🔗 관련 이슈
close #
🔄 변경 사항
변경된 내용
변경된 내용에 대한 자세한 설명을 작성해주세요.
변경 이유
왜 이러한 변경이 필요했는지 설명해주세요.
✅ 체크리스트