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] #67 유저 스터디룸 목록 조회 API #74

Merged
merged 22 commits into from
Aug 12, 2024

Conversation

karryred
Copy link
Collaborator

@karryred karryred commented Aug 10, 2024

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

반영 브랜치

feature/#67-user-studyroom/GAJI-98 -> develop

  • 변경 사항

  • QueryDSL 의존성을 추가하였습니다.
  • RoomTypeEnum을 추가하였습니다.

이슈

  • CheckPage Validator를 추가하였는데 커서페이징을 사용하여 쓸 기회가 없어 추후에 검증해보겠습니다.

테스트 결과

-성공 시
룸 가져오기

-더 가져올 게 있을 시
더 가져올게 있을 시,

-커서 id를 지정해서 더 가져올 시(무한 스크롤)
커서 id 지정

@karryred karryred added ✨ feature 새로운 기능 구현 ⚙ Setting 개발 환경 세팅 labels Aug 10, 2024
@karryred karryred self-assigned this Aug 10, 2024

LocalDate now = LocalDate.now();

BooleanExpression cursorCondition = (room.studyStartDay.eq(cursorDate).and(room.id.gt(cursorId)))
Copy link
Collaborator

Choose a reason for hiding this comment

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

중복되는 부분은 메서드화 하면 좋을 것 같습니다!

Copy link
Collaborator

@mmingoo mmingoo left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

Copy link
Collaborator

@spenshark spenshark left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@karryred karryred merged commit ccc7810 into develop Aug 12, 2024
mmingoo pushed a commit that referenced this pull request Dec 19, 2024
✨[feature] #67 유저 스터디룸 목록 조회 API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙ Setting 개발 환경 세팅 ✨ feature 새로운 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants