Conversation
xjvmdutl
reviewed
Mar 7, 2025
src/main/java/com/retrip/crew/application/in/response/CrewDetailResponse.java
Outdated
Show resolved
Hide resolved
xjvmdutl
reviewed
Mar 7, 2025
| private CrewMembers crewMembers; | ||
|
|
||
| @Column(name = "max_members", nullable = false) | ||
| private int maxMembers; |
Collaborator
There was a problem hiding this comment.
해당 필드를 이 이슈에서 추가한 이유가 있을까요??
작업 내용에는 없는것 같아서.. 만약 추가된다면 작업 내용에 스토리 적는게 좋을거 같습니다.
Contributor
Author
There was a problem hiding this comment.
크루도 최대 받을수 있는 인원이 정해져있는데 크루 리스트 조회할때 사용자가 최대인원과 현재 가입되어있는 인원 ex ) 2/10 이렇게 보여야하기에 필요해서 추가했습니다
| } | ||
|
|
||
| /** 무한스크롤 위해 hasNext 체크 및 제거하는 util */ | ||
| public static <T> Slice<T> checkEndPage(Pageable pageable, List<T> results) { |
Collaborator
There was a problem hiding this comment.
해당 코드 다음 회의때 공유해주실수 있을까요?
무한 스크롤 방지 같은 경우에는 다른 도메인에도 사용될 수 있고, 공통으로 사용하면 좋을 거 같아서 공유하면 좋을거 같아요!
참고한 코드가 있으면 링크를 공유주셔도 됩니다
Contributor
|
크루 리스트 조회와 크루 상세 조회 기능을 개발하신 것 같네요. |
mandykr
reviewed
Mar 7, 2025
src/main/java/com/retrip/crew/application/out/repository/CrewRepository.java
Outdated
Show resolved
Hide resolved
...n/java/com/retrip/crew/infra/adapter/out/persistence/mysql/query/CrewQuerydslRepository.java
Outdated
Show resolved
Hide resolved
src/test/java/com/retrip/crew/common/helper/CrewTestHelper.java
Outdated
Show resolved
Hide resolved
src/main/java/com/retrip/crew/infra/adapter/in/presentation/rest/CrewController.java
Show resolved
Hide resolved
Co-authored-by: junhokim <junhokim@gmarket.com>
pparkjs
force-pushed
the
feature-4/look-up-crew
branch
from
32083ec to
d3a480c
Compare
ekfrehd
pushed a commit
that referenced
this pull request
Jun 3, 2025
* [feat] 크루 리스트 조회 API 기능 구현 및 테스트코드 리팩토링 (#4) * [feat] 크루 상세 조회 API 기능 구현 (#4) * feat: 스웨거 추가 작업 (#24) Co-authored-by: junhokim <junhokim@gmarket.com> * [fix] 크루 상세 조회 API 코드리뷰 반영 (#4) * [fix] 단위테스트 SpringBootTest -> DataJpaTest 사용하도록 수정 (#4) --------- Co-authored-by: Kimjunho <39546306+xjvmdutl@users.noreply.github.com> Co-authored-by: junhokim <junhokim@gmarket.com>
ekfrehd
added a commit
that referenced
this pull request
Jun 29, 2025
* feat: PR 템플릿 and PR Discord 알림 연동 * feat: Crew 생성 (#20) * feat: Crew 생성 * feat: Code 리뷰 반영, Exception 추가 * feat: 주석 제거 및 코드 수정 --------- Co-authored-by: junhokim <junhokim@gmarket.com> * feat: 비즈니스 예외와 API 공통 응답 모듈 추가 (#23) * feat: 비즈니스 예외와 API 공통 응답 모듈 추가 - vo 검증에서 발생하는 비즈니스 예외 삭제 및 InvalidValueException 으로 변경 * feat: 스웨거 추가 작업 (#24) Co-authored-by: junhokim <junhokim@gmarket.com> * [feat] 크루 상세, 리스트 조회 API 기능 구현 (#25) * [feat] 크루 리스트 조회 API 기능 구현 및 테스트코드 리팩토링 (#4) * [feat] 크루 상세 조회 API 기능 구현 (#4) * feat: 스웨거 추가 작업 (#24) Co-authored-by: junhokim <junhokim@gmarket.com> * [fix] 크루 상세 조회 API 코드리뷰 반영 (#4) * [fix] 단위테스트 SpringBootTest -> DataJpaTest 사용하도록 수정 (#4) --------- Co-authored-by: Kimjunho <39546306+xjvmdutl@users.noreply.github.com> Co-authored-by: junhokim <junhokim@gmarket.com> * 크루 참여 요청 (#29) * feat: 크루 모집상태 변경시 최대 인원수 검증 추가 - IllegalStateException 추가 - common exception 생성자 수정 * refactor: CrewMembers size 조회 메서드 추가 * fix: conflict repair * feat:크루 참여 요청 질문 등록 * feat:크루 참여 요청 질문 등록 * [FEAT]-recruitment로 연관관계 변경 * 크루 참여 요청 질문 등록 수정 * 크루 참여 요청 질문 등록 수정 구현 --------- Co-authored-by: junhokim <junhokim@gmarket.com> Co-authored-by: Kimjunho <39546306+xjvmdutl@users.noreply.github.com> Co-authored-by: mandykr <94955454+mandykr@users.noreply.github.com> Co-authored-by: coPpark <qkrwjdtn963@gmail.com> Co-authored-by: MonChest <p_caso@atzsoft.kr>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔍 PR 타입 선택
아래 타입 중 해당하는 하나를 선택해 주세요. 반드시 하나만 선택해 주세요.
feat: 새로운 기능 추가fix: 버그 수정docs: 문서 수정style: 코드 포맷팅, 세미콜론 누락, 코드 변경이 없는 경우refactor: 코드 리팩토링test: 테스트 코드 추가 또는 수정chore: 빌드 업무 수정, 패키지 매니저 수정 등 기타 작업📝 변경 사항 요약
🛠 관련 이슈
Resolves: #4
Ref: #4
Related to:
close: #4
추가 설명 (선택 사항)
변경 사항에 대한 추가 설명을 작성해 주세요.