feat: 플랜 및 캘린더 조회 기능 추가

[junseoparkk]

📝 개요

• 새로운 플랜 기능을 추가합니다.
• 캘린더 조회 기능을 추가합니다.

✨ 변경 사항

• ✨ schdule, calendar 패키지 추가
• ✨ 캘린더 조회를 위한 그룹 로직 리팩토링

🔗 관련 이슈

• closed feat: 플랜 및 캘린더 조회 기능 추가 #255

ℹ️ 참고 사항

• 우선 기능 구현이 너무 늦은점 정말 죄송합니다.
• 기존 플랜과 네이밍이 겹쳐 schedule 이라는 네이밍을 사용하였습니다. core 에선 네이밍이 겹쳐도 패키지가 달라 괜찮지만, 컨트롤러나 JPA 관련 클래스에선 빈이 중복 등록되는 등의 문제가 있었습니다.
• 기능 구현을 최우선으로 하여 최적화, 메서드 리팩토링, 논리적 검증, 테스트, API 스펙 등 개선해야할 것들이 많다고 생각합니다. 저번 회의때 말씀하신 것처럼 스프린트 주기를 빠르게 하여 처리 가능한 것부터 우선순위를 두어 고쳐나갈 예정입니다.
• 패키지 구조는 기존에 제가 생각했던 것처럼 3개로 분리하였습니다. 우선 돌아가는 코드를 만들고 패키지 통합이나 분리가 맞다고 판단했고, 이후 API 스펙은 프론트엔드 개발자와 협의하여 빠르게 수정할 수 있기 때문에 core 쪽을 중점으로 봐주시면 감사할것같습니다.
• 해당 PR 에 내용은 모두 담기엔 양이 많은 것 같아서 정규 회의나 백엔드 회의 때 하나씩 수정 사항이나 방향성을 이야기해보면 좋을 것 같습니다.

[github-actions]

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 4b154fd.

♻️ This comment has been updated with latest results.

[JiHongKim98]

사실 회의할때 bounded context 기반으로 패키지를 분리할건지 aggregate root 기반으로 패키지를 구분할건지 준투 의견을 따른다고 이야기 했었는데, 현재 PR 리뷰를 진행하면서 느낀게 동일 bounded context 내의 aggregate root 로직들이 서로 상관되어 있는 경우가 생각보다 너무 많은 것 같아서 이거는 처음부터 합치고 가는게 좋을 것 같아요.. (스노우볼 방지..)
ex) SchedulePlanCommandService 에서의 validateGroupMember() 같은 부분 처럼 연관되어 있는 경우

그리고 mapper 로직은 mapper 클래스 안에 있는게 좋을 것 같아요..!
(코드 파편화가 점점 커다란 스노우볼이 되고 있어서 나중에 리팩토링이 어려울 것 같네유)

(코드 리뷰는 일단 인프라쪽 제외하고 진행했습니다. 수정 커밋 올라오면 인프라쪽도 한번에 리뷰할게요!)