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

[Feature/402] 모임 목록 조회시에 기록 존재 여부 필드 -> 기록or활동 존재 여부로 변경 #403

Merged
merged 3 commits into from
Nov 1, 2024

Conversation

joowojr
Copy link
Contributor

@joowojr joowojr commented Oct 30, 2024

Type of change

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

PR Desciption

변경 사항 설명

  • 필드명 hasDiary -> hasRecord로 수정
  • ScheduleSummaryQuery.java : activityId 필드 추가
    쿼리에서 activity 테이블을 schedule 테이블과 LEFT OUTER JOIN으로 연결하여 activity id로 activity의 존재 여부를 반환하도록 변경하였습니다.

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

  • 개선할 수 있는 부분이 있을지 봐주시면 좋을 것 같습니다.

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

관련 이슈

@github-actions github-actions bot added the ✨ feature 기능 구현 관련 label Oct 30, 2024
@joowojr joowojr requested a review from hosunglee222 October 30, 2024 11:28
Copy link
Member

@hosunglee222 hosunglee222 left a comment

Choose a reason for hiding this comment

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

LGTM!

@hosunglee222 hosunglee222 merged commit 23d7d57 into dev Nov 1, 2024
5 checks passed
@hosunglee222 hosunglee222 deleted the feature/402 branch November 1, 2024 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 기능 구현 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 모임 목록 조회시에 기록 존재 여부 필드 -> 기록or활동 존재 여부로 변경
2 participants