Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Refactor] 기록 캘린더 조회 변경 #394

Closed
1 of 10 tasks
hosunglee222 opened this issue Oct 26, 2024 · 0 comments · Fixed by #395
Closed
1 of 10 tasks

[Refactor] 기록 캘린더 조회 변경 #394

hosunglee222 opened this issue Oct 26, 2024 · 0 comments · Fixed by #395
Assignees
Labels
♻️ refactor 리펙터링 관련

Comments

@hosunglee222
Copy link
Member

Type of change

  • Feature : 새로운 기능 추가
  • Bug fix : 버그 수정
  • Refactor : 코드 리팩토링 작업
  • Document : 문서작업
  • Test : 테스트 코드 작성 및 테스트 작업
  • Style : 코드 스타일 및 포맷팅 작업
  • CI/CD : CI/CD 작업 수정
  • Chore : 패키지 매니저, 라이브러리 업데이트 등의 작업

Description

  • 기록 캘린더 조회 API 를 변경합니다.
    • 기존 모든 기록에대한 날짜 Return에서 개인 스케줄에 대한 기록과 모임 스케줄에 대한 기록을 분리합니다.
    • 사유 : UI 상 스케줄에 따른 기록의 색깔 차이

Todo

  • DTO 재구조화
  • 로직 수정

Reference(Optional)

image
@hosunglee222 hosunglee222 added the ♻️ refactor 리펙터링 관련 label Oct 26, 2024
@hosunglee222 hosunglee222 self-assigned this Oct 26, 2024
hosung-222 added a commit that referenced this issue Oct 26, 2024
hosung-222 added a commit that referenced this issue Oct 26, 2024
<body>
도메인 관련 로직을 위임하여 재사용성 강화

- 관련 : #394
hosung-222 added a commit that referenced this issue Oct 26, 2024
<body>
변경된 DTO에 맞게 타입별로 convert

- 관련 : #394
hosung-222 added a commit that referenced this issue Oct 26, 2024
hosung-222 added a commit that referenced this issue Oct 26, 2024
<body>
생일 스케줄에 대한 기록 날짜를 반환하기 위한 컬럼 추가

- 관련 : #394
hosung-222 added a commit that referenced this issue Oct 26, 2024
<body>
TreeSet 사용으로 자동 정렬 및 중복 제거
Stream API의 복잡성 제거
EnumMap 사용으로 타입 안전성 확보

- 관련 : #394
@hosunglee222 hosunglee222 linked a pull request Oct 26, 2024 that will close this issue
7 tasks
@hosung-222 hosung-222 self-assigned this Oct 26, 2024
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 a pull request may close this issue.

2 participants