feat: 지역으로 북토크 리스트 조회 #35
Merged
Conversation
This file contains 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
dong2ast
reviewed
Jul 11, 2023
Comment on lines
+24
to
+41
| City city = cityRequestDto.getCity(); | ||
| List<Place> placeList; | ||
|
|
||
| if (city.equals(City.UIJEONGBU_SI)) { | ||
| placeList = placeRepository.findAll(); | ||
| } else { | ||
| placeList = placeRepository.findAllByCity(city); | ||
| } | ||
|
|
||
| List<BooktalkResponseDto> booktalkList = new ArrayList<>(); | ||
| placeList.forEach(place -> { | ||
| place.getBooktalkList().forEach(booktalk -> { | ||
| // 모집중인 북토크만 추가 | ||
| if (booktalk.getBooktalkStatus() == BooktalkStatus.RECRUITING) { | ||
| booktalkList.add(BooktalkResponseDto.of(booktalk)); | ||
| } | ||
| } | ||
| ); |
There was a problem hiding this comment.
서비스 단이 잘 이해가 안가서 그런데 place를 조회하고 그 place에 있는 북토크들 조회한 후, 그 북토크들 중 모집 중인 것만 리스트에 추가해서 반환하는 걸까요?
기능상으로는 선택한 지역에 Recruiting인 북토크를 조회하면 되는건가요?
There was a problem hiding this comment.
맞습니다! 검색한 지역내에 모집중인 북토크를 마감임박순으로 정렬해서 보여주는 기능입니다
dong2ast
approved these changes
Jul 11, 2023
dong2ast
pushed a commit
that referenced
this pull request
Jul 11, 2023
* modify: 지역으로 공간 조회 RequestBody CityRequestDto로 통일 * feat: 지역으로 북토크 리스트 조회 * fix: @transitional 추가 * fix: url과 Valid 설정
dong2ast
added a commit
that referenced
this pull request
Jul 11, 2023
* add: BooktalkCreateRequestDto, ResponseDto 추가 * feat: POST 북토크 생성 구현 * add: BooktalkUpdateDto 추가 * modify: Booktalk에 patchBooktalk추가 * feat: PATCH 북토크 수정하기 구현 * fix: 북토크 생성 시 member와 place 연결 * add: BooktalkDeleteResponseDto 추가 * modify: Author와 Booktalk에 deleteBooktalk 추가 * feat: DELETE 북토크 삭제 구현 * feat: GET 북토크 상세조회 구현 * refactor: 원시타입 제거 * refactor: access level 수정 * refactor: 원시타입 제거 * fix: 리팩터 오류 수정 * feat: 지역으로 북토크 리스트 조회 (#35) * modify: 지역으로 공간 조회 RequestBody CityRequestDto로 통일 * feat: 지역으로 북토크 리스트 조회 * fix: @transitional 추가 * fix: url과 Valid 설정 * feat: 가장 가까운 북토크 멤버 필드에 추가 * feat: 예정된 북토크 조회 메서드 추가 * chore: 원시 타입 수정 오류 해결 * fix: 예정된 북토크 조회해오는 메서드 수정 * add: initDB에 데이터 추가 * add: initDB에 데이터 추가 * chore: Booktalk의 멤버 변수 -> author 이름으로 변경 * chore: Booktalk의 멤버 변수 -> author 이름으로 변경 * refactor: Member가 가진 Author 필드를 AuthorProperty로 이름 변경 작업 * feat: 현재 Issue에 대한 테스트 데이터 InitDb에 추가 * fix: BooktalkResponseDto에 image String도 추가 --------- Co-authored-by: onpyeong <[email protected]> Co-authored-by: Suhyeon <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
📌 관련 이슈
close #31
✨ 과제 내용
해당 공간에 BooktalkList에서 현재 모집중인 북토크만 모아서 마감 임박순으로 정렬하여 조회하도록 구현했습니다!
📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들