-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Develop] 3주차 과제 PR #35
Merged
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
이외 토큰 없이 공통으로 조회 가능한 기능은 추후 구현 예정
Springdoc 2.0.3 버전 종속성 추가
성공 및 실패 사례는 아직 미처리, 추후 추가 예정
sort 객체를 이용하는 방법에서 메서드 이름으로 지정하는 방식으로 변경
동일 리소스에 대한 Mapping 가독성 상승시키기
# Conflicts: # build.gradle # src/main/java/team7/inplace/video/application/VideoService.java # src/main/java/team7/inplace/video/presentation/VideoController.java
Place 변경사항 Merge 이후 구현
cookie에 있는 token을 검증하는 로직 관련 이슈: #2
…tep3/Team7_BE into feat/#2-authorization
현재 로그인한 user의 username을 가져오는 기능 관련 이슈:#2
테스트는 도메인 별로 패키지를 만들고 패키지 명은 도메인 명과 동일하게 하기
로그인한 유저의 userId는 Authorization.getUserId()로 가져옵니다 관련 이슈: #2
리소스 명 video에서 videos로 수정
테스트 과정에서 의도한 바와 다르게 동작하는 것을 발견하고 수정
global exception으로 한번에 사용하기로 정했었고, null 사용을 최소화 하기 위해 dto를 추가하였습니다. 관련 이슈: #2
클래스에 정의된 상수 사용을 static import 방식으로 변경
…tep3/Team7_BE into feat/#2-authorization
Feat/#7 place - 카테고리, 인플루언서 필터링을 적용시켰어요!
…rization [Feat] #2 authorization을 구현했어요.
…swaggerAndConvention [Refactor] #29 Swagger 코드 스타일과 패키지 네이밍 컨벤션을 맞춰 보아요
…tep3/Team7_BE into feat/#9-summarize-youtube
[Feat] #8 CI/CD 관련 수정했어요
…uencer [Feat] #20 로그인 없을 때 인플루언서 반환을 구현했어요
…rize-youtube [Feat]#9 유튜브에서 장소 정보를 크롤링해요
[Weekly] 2주차 과제 PR
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.
✨ 작업 내용
✨ 참고 사항
⏰ 현재 버그
✏ Git Close