-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: 회원 관련 API 구현 및 CI 관련 워크플로우 추가 #20
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
- 페이징은 회원 검색 뿐만 아니라 여러 곳에서 쓰이기 때문에 `PagingUtil`이라는 클래스를 따로 만듦
- S3 파일 업로드도 구현 완료
- AWS S3 민감한 정보를 yml 파일에 안전하게 주입하는 스크립트도 추가함
Test Results18 tests 18 ✅ 1s ⏱️ Results for commit 0aa036d. ♻️ This comment has been updated with latest results. |
peeerr
force-pushed
the
feature/19-member
branch
from
September 26, 2024 14:04
66914f7
to
0aa036d
Compare
7 tasks
HyeJiJUN11
approved these changes
Sep 27, 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.
📝 작업 내용
1. 회원 관련 API 구현 및 테스트 코드 작성
PagingUtil
테스트 코드 작성2. Weekly PR 시 자동 CI 테스트 워크플로우 추가
3. SuccessResponse 편의 메서드 추가
현재 SuccessResponse 생성 코드가 너무 반복되고 있어, 메서드 추가해 두었습니다.
code
가 200이 아니고info
응답이 필요하다면 맨 위 메서드,code
가 200이고,info
응답이 필요하다면 가운데 메서드, 필요 없다면 아래 메서드를 사용해 주시면 됩니다!4. 환경별 설정 파일 분리
파일은 이렇게 dev와 prod(운영) 환경으로 나눴습니다
dev에는 민감한 정보도 담겨 있어서 .gitignore에 등록해두었습니다
기존
application.yml
은 제거해 주시고, 아래와 같이application-dev.yml
을 따로 만들어 주세요!secret에 관한 정보는 따로 드리겠습니다!
그리고, 기본 액티브 프로파일이 prod라서 서버 기동 시 오류가 발생할 텐데 아래와 같이 설정해 주세요
5. 페이징 관련
저는 커서 기반 페이징을 저번에 코드리뷰 드렸을 때의 Spring Data JPA의 Scrolling using Keyset-Filtering 방식으로 작업을 진행했는데,
페이징 작업이 계속 반복될 것 같아서
PagingUtil
이라는 클래스를 만들어 두었습니다.이 방식도 한번 고려해 주세요!
🔗 관련 이슈
close #19