Conversation
* refactor: Stamp deadline 필드 제거 * refactor: StampService createStamp(), createStamps(), updateStampNameAndDeadline() 메서드 리팩토링 * refactor: StampRepository findAllByTripIdOrderByDeadlineAsc() 메서드 리팩토링 * refactor: StampFactory create() 메서드 리팩토링 * refactor: StampPolicy validateStampDeadline() 메서드 제거 * refactor: StampErrorCode deadline 관련 에러코드 제거 * refactor: CreateStampRequest, UpdateStampRequest 요청 DTO deadline 필드 제거 * refactor: LoadStampDetailResponse, LoadStampInfoResponse 응답 DTO deadline 필드 제거 * refactor: StampInfo deadline 필드 제거 * test: StampFixture 클래스 deadline 필드 제거 * test: CreateStampRequestFixture 클래스 deadline 필드 제거 및 리팩토링 * test: StampService 단위 테스트에서 스탬프 마감일 관련 테스트 제거 * test: TripController, StampController 통합 테스트에서 스탬프 마감일 관련 테스트 제거
chaiminwoo0223
approved these changes
Aug 11, 2025
Contributor
chaiminwoo0223
left a comment
chaiminwoo0223
left a comment
There was a problem hiding this comment.
고생하셨습니다. 머지 부탁드립니다!
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
2 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.
📌 작업 내용 및 특이사항
✅ 스탬프 엔티티 마감일(deadline) 필드 제거
마감일(deadline)필드를 제거했습니다.✅ 스탬프 마감일 관련 로직 리팩토링
StampFactory클래스에서 스탬프를 생성하는create()메서드의 매개변수에서deadline을 제거했습니다.StampPolicy클래스에서 스탬프의 마감일을 검증하는validateStampDeadline()메서드를 제거했습니다.StampErrorCode이넘 클래스에서 스탬프의 마감일의 과거 에러코드와 여행 마감일과 비교해 과거일 경우 에러코드를 제거했습니다.StampService.updateStampNameAndDeadline()메서드를updateStampName()으로 변경하고, 로직을 리팩토링 했습니다.StampService.createStamp(),StampService.createStamps()메서드에서 스탬프 마감일을 검증하는 로직을 제거했습니다.마감일(deadline)필드로 오름차순 정렬하는 쿼리 메서드StampRepository.findAllByTripIdOrderByDeadlineAsc()를findAllByTripIdOrderByCreatedAtAsc()로 변경해 생성일 필드를 기준으로 오름차순 정렬해 조회되도록 리팩토링했습니다.CreateStampRequest요청 DTO의deadline필드를 제거했습니다.UpdateStampNameAndDeadlineRequest요청 DTO의 이름을UpdateStampRequest로 수정하고deadline필드를 제거했습니다.LoadStampDetailResponse,LoadStampInfoResponse응답 DTO의deadline필드를 제거했습니다.application.dtoStampInfo클래스의deadline필드를 제거했습니다.✅ 스탬프 마감일 관련 테스트 코드 수정
StampFixture클래스의 마감일 관련 코드를 제거했습니다.CreateStampRequestFixture,UpdateStampRequestFixture클래스에서deadline필드를 제거했습니다.StampServiceTest에서 스탬프 마감일 관련 검증 테스트 코드를 제거했습니다.StampController,TripController통합 테스트에서 스탬프 마감일 관련 예외 테스트 코드를 제거했습니다.🌱 관련 이슈
🔍 참고사항(선택)
📚 기타(선택)