[Feat] #6 로그인 없이 공통으로 조회할 수 있는 다양한 조건의 비디오 조회 기능을 구현했어요! #21
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
added 3 commits
September 26, 2024 22:54
이외 토큰 없이 공통으로 조회 가능한 기능은 추후 구현 예정
Springdoc 2.0.3 버전 종속성 추가
성공 및 실패 사례는 아직 미처리, 추후 추가 예정
BaeJunH0
changed the title
sanghee0820
requested changes
Sep 27, 2024
src/main/java/team7/inplace/video/application/VideoService.java
Outdated
Show resolved
Hide resolved
src/main/java/team7/inplace/video/presentation/VideoController.java
Outdated
Show resolved
Hide resolved
| // 토큰 불필요 메서드 | ||
| // 내 주변 장소 반환 기능 ( PR 존재 ) weekly에 merge 시 개발 | ||
| @GetMapping("/video") | ||
| @Operation(summary = "내 주변 그 곳", description = "내 위치를 기준으로 Video 정보를 조회합니다.") |
There was a problem hiding this comment.
저는 개인적으로 인터페이스에서 Swagger에 대한 명세를 한 후 Implements해주는걸 좋아하는데 어떻게 생각하시나요?
There was a problem hiding this comment.
목요일 회의 시간에 어떻게 하는 지 알려주시겠어요? 딱 읽기만 해서는 감이 오질 않네요 ㅎㅎ...
added 6 commits
September 28, 2024 23:39
sort 객체를 이용하는 방법에서 메서드 이름으로 지정하는 방식으로 변경
동일 리소스에 대한 Mapping 가독성 상승시키기
# Conflicts: # build.gradle # src/main/java/team7/inplace/video/application/VideoService.java # src/main/java/team7/inplace/video/presentation/VideoController.java
|
Weekly/2 브랜치 머지 이후 리펙터링 작업 완료했습니당 |
added 3 commits
September 29, 2024 00:29
Place 변경사항 Merge 이후 구현
테스트는 도메인 별로 패키지를 만들고 패키지 명은 도메인 명과 동일하게 하기
added 7 commits
October 1, 2024 15:44
리소스 명 video에서 videos로 수정
테스트 과정에서 의도한 바와 다르게 동작하는 것을 발견하고 수정
헤더를 분석하여 토큰의 유무를 확인하는 로직 구현
|
토큰 검증 로직에 대해서는 목요일에 이야기 나누어용 |
sanghee0820
requested changes
Oct 1, 2024
src/main/java/team7/inplace/video/application/VideoService.java
Outdated
Show resolved
Hide resolved
added 2 commits
October 1, 2024 22:27
application yaml 파일에서 import 하도록 변경
sanghee0820
approved these changes
Oct 2, 2024
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.
✨ 작업 내용
"새로 등록된 그 곳" API 작성 및 이를 위한 id 기준 내림차순 정렬한 비디오 정보 조회 기능 구현
"내 주변 그 곳" API 작성 및 이를 위해 Place 엔티티를 기준으로 비디오 정보 조회 기능 구현
Swagger를 사용하기 위한 종속성 추가 및 Swagger를 사용한 API 설명 추가
✨ 참고 사항
구현하지 못한 API 들은 다른 기능에 대한 종속성으로 인해 구현하지 못했음 ( Place, 조회수 )
Place에 대한 부분 ( "내 주변 그곳" ) 은 Merge 되는 대로 구현 예정조회수에 대한 부분 ( "쿨한 그 곳" ) 역시 구현되는 대로 구현 예정
테스트 코드 작성 예정
비디오 조회 시 페이징을 위한 조건들은 받지만 List 형식의 반환 ( 어차피 PlaceRepository의 메서드를 사용하여 조회 시, 페이지 크기 만큼의 엔티티만 조회되므로 쿼리 사용이 빈번하지 않을 것으로 생각해서 )
⏰ 현재 버그
✏ Git Close