Skip to content

Conversation

@leebs0521
Copy link
Collaborator

@leebs0521 leebs0521 commented Nov 25, 2024

resolved :

📌 과제 설명

  • 봉사 모집글 수정 기능
    • 모집 정보 수정
    • 모집 위치 수정

👩‍💻 요구 사항과 구현 내용

  • RecruitBoard 안에 VolunteerInfo 객체를 두어 봉사 모집 정보 관리
  • 위치 조회, 수정 기능
  • 봉사 모집글 조회, 수정 기능
  • 논리 삭제 메서드 작성 및 테스트 (RecruitBoard) 적용
  • LocalDateTimeFixture 으로 고정된 LocalDateTime 객체 생성

✅ PR 포인트 & 궁금한 점

  • 에러 메세지 ENUM 변경은 PR 따로 올려서 진행하겠습니다.
  • 테스트가 적절한지 확인해주시면 감사하겠습니다.
    • RecruitBoard의 시작 시간 종료시간 업데이트는 추후 작성하겠습니다.

@leebs0521
Copy link
Collaborator Author

CI 테스트용으로 PR 올렸고 수정해야할 부분 더 있어서 일단 닫겠습니다.

@leebs0521 leebs0521 closed this Nov 25, 2024
Copy link
Collaborator

@m-a-king m-a-king left a comment

Choose a reason for hiding this comment

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

테스트 코드에서 감탄과 경악을 함께 느끼고 갑니다. 너무 수고하셨습니다.

전체적으로 클래스 마지막에 개행이 빠진 부분을 수정하셔야 할 것 같습니다. 서진 님이 오시기 전에...

Copy link
Collaborator

@ayoung-dev ayoung-dev left a comment

Choose a reason for hiding this comment

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

수고하셨습니당

@sonarqubecloud
Copy link

@leebs0521 leebs0521 merged commit 21e76f5 into main Nov 26, 2024
2 checks passed
@leebs0521 leebs0521 deleted the feature/26-add-recurit-board-update branch November 26, 2024 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants