-
Notifications
You must be signed in to change notification settings - Fork 0
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: 숙소 조회 시, 객실 최대 인원과 최대 page count를 응답에 포함해 보내는 API를 구현합니다. #104
Conversation
fix: ssh 키 수정
feat: 상품 전체 조회 및 상세 조회 기능 구현
…into feature/product
- 총 예약 인원 'totalPeople', 방문 시작일 'startDate', 방문 마지막일 'endDate' 필드 추가
- 테이블 여부 컬럼인 table이 예약어여서 변경
돼 테스트가 안되는 점 수정
- Open API에서 받아온 데이터에 맞게 자료형 수정
feat: Open API에서 더 많은 정보를 활용하도록 구현
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고 많으셨습니당~! 🐻 몇 가지 코멘트 남겨뒀으니 확인 후 수정 부탁드려요!!
QueryDSL 같은 경우는 추후 수정하셔도 되니 어떻게 하실 것인지 결정하신 후 답변 달아주세요~~
@Query(value = "SELECT p FROM Product as p " | ||
+ "JOIN FETCH p.rooms as r " | ||
+ "JOIN FETCH p.address as addr " | ||
+ "WHERE p.name LIKE %:#{#searchKeywordRequest.productName}% " | ||
+ "AND addr.detailAddress LIKE %:#{#searchKeywordRequest.address}% " | ||
+ "AND p.category in :#{#searchKeywordRequest.category} " | ||
+ "AND r.capacity >= :#{#searchKeywordRequest.capacity}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QueryDSL을 통해 Java 문법으로 쿼리를 날릴 수 있습니다! 마침 저희 프로젝트에서도 사용하도록 설정되어 있으니, 사용해보시는 게 좋을 것 같습니다~~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
설정도 되있으니 QueryDSL 사용해서 하겠습니다!
src/test/resources/application.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
secret 값들이 모두 올라가 있습니다..! 해당 커밋 남지 않도록 수정해주세요!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오호...저게 왜 올라갔지 수정하겠습니다.
8e5f4f5
to
b849287
Compare
Close #103 을 참고해주세요!
💡 Motivation
📌 Changes
🫱🏻🫲🏻 To Reviewers