Skip to content

Comments

refactor: Mission 순서, 메모 필드 제거 및 리팩토링(#53)#62

Merged
hisonghy merged 1 commit intodevelopfrom
refactor/53
Aug 16, 2025
Merged

refactor: Mission 순서, 메모 필드 제거 및 리팩토링(#53)#62
hisonghy merged 1 commit intodevelopfrom
refactor/53

Conversation

@hisonghy
Copy link
Contributor

📌 작업 내용 및 특이사항

✅ Mission 엔티티 missionOrder(순서), memo(메모) 필드 제거

  • 요구사항에 따라 Mission 엔티티의 missionOrder(순서), memo(메모) 필드를 제거했습니다.

✅ Mission missionOrder(순서), memo(메모)와 관련된 로직 제거

  • Mission을 생성하는 Mission.of(), MissionFactory.create() 메서드에서 missionOrder, memo 필드를 제거했습니다.
  • Mission을 수정하는 Mission.update() 메서드명을 updateName()으로 변경하고, name 필드만 수정하도록 리팩토링했습니다.
  • MissionFacade, MissionService의 미션의 이름과 메모를 수정하는 updateMissionNameAndMemoIfPresent() 메서드를 updateMissionNameIfPresent()로 변경했습니다.
  • 미션의 순서를 변경하는 API 관련 로직을 모두 제거했습니다. (MissionController.updateMissionOrders, MissionFacade. updateMissionOrders, MissionService.updateMissionOrders)
  • 미션의 순서를 기준으로 정렬하는 쿼리 메서드(MissionRepository.findAllByStampIdAndDeletedAtIsNullOrderByMissionOrder)를 createdAt 기준으로 정렬하도록 변경하고, 미션 순서와 관련된 쿼리 메서드를 모두 제거했습니다. (MissionRepository.findAllByStampIdOrderByMissionOrder, MissionRepository.existsByStampIdAndMissionOrderAndDeletedAtIsNull)
  • 미션의 순서와 관련된 검증 메서드, 에러코드를 모두 제거했습니다.

✅ 미션 생성/수정 요청 DTO name 필드명 변경

  • 요구사항에 따라 CreateMissionRequest, UpdateMissionRequest DTO 클래스의 name 필드명을 missionName으로 변경했습니다.

✅ 테스트

  • 미션의 순서, 메모 필드와 관련된 모든 테스트 코드를 제거, 리팩토링 했습니다.

🌱 관련 이슈


🔍 참고사항(선택)


📚 기타(선택)

* refactor: Mission 엔티티 순서, 메모 필드 제거
* refactor: Mission 생성/수정, 쿼리 메서드, 검증/에러코드, DTO 리팩토링

* test: Mission 순서, 메모 관련 통합/단위 테스트 코드 리팩토링
@hisonghy hisonghy self-assigned this Aug 16, 2025
@hisonghy hisonghy added the 🪄refactor 기능 개선 및 리팩토링 label Aug 16, 2025
Copy link
Contributor

@chaiminwoo0223 chaiminwoo0223 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

요청/응답 DTO 모두 동일한 필드를 사용하도록 하여 도메인 명확성이 높아진 것 같습니다. 이후 리팩토링 단계에서 다른 DTO들도 함께 개선하면 좋을 것 같아요.

@chaiminwoo0223
Copy link
Contributor

고생하셨습니다. 머지 부탁드립니다!

@hisonghy hisonghy merged commit 137deba into develop Aug 16, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪄refactor 기능 개선 및 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🪄[REFACTOR]: 미션 엔티티 순서, 메모 필드 제거

2 participants