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

feat : 그룹 캡슐 목록 조회 #476 #489

Merged
merged 10 commits into from
Jun 12, 2024

Conversation

seokho-1116
Copy link
Collaborator

작업 내용 (Content)

  • 그룹 캡슐 목록 조회 기능 추가
  • 테스트 추가

링크 (Links)

기타 사항 (Etc)

  1. 그전에 쿼리로 그룹원 여부 확인 및 검증
    Screenshot from 2024-06-08 21-00-36
  2. join + where로 검증
    Screenshot from 2024-06-08 21-04-02
  • 그룹에 대한 정보를 조회할 때 그전에 그룹원인지 검증하고 싶은데 뭐가 더 좋을지 모르겠음.
    1은 한방 쿼리로 해결할 수 있긴하나 그룹원이 아닌데 그룹 정보를 조회하는 API에서 그룹원이 아닌데 조회가 된다는 것이 약간 논리적으로 이상하긴함

현재는 1번을 택해서 구현했음.

먼저 앞에 브랜치를 머지해버려서 다시올릴께;;

Merge 전 필요 작업 (Checklist before merge)

희망 리뷰 완료 일 (Expected due date)

@seokho-1116
Copy link
Collaborator Author

@GaBaljaintheroom 이거 머지 순서 틀려서 다시 승인해주라 ㅜ 쏘리

…#479

# Conflicts:
#	backend/core/src/main/java/site/timecapsulearchive/core/domain/capsule/group_capsule/api/GroupCapsuleApi.java
#	backend/core/src/main/java/site/timecapsulearchive/core/domain/capsule/group_capsule/service/GroupCapsuleService.java
#	backend/core/src/test/java/site/timecapsulearchive/core/common/fixture/domain/CapsuleFixture.java
#	backend/core/src/test/java/site/timecapsulearchive/core/domain/capsule/group_capsule/repository/GroupCapsuleQueryRepositoryTest.java
#	backend/core/src/test/java/site/timecapsulearchive/core/domain/capsule/group_capsule/service/GroupCapsuleServiceTest.java
@GaBaljaintheroom GaBaljaintheroom merged commit 752cbd3 into develop_back_core Jun 12, 2024
2 checks passed
@seokho-1116 seokho-1116 deleted the refact/common_dto-B-#479 branch June 17, 2024 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants