main push #73
Merged
main push #73
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
- s3에 image 파일 저장 로직 분리 - img -> images로 통일 - todo - 여전히 중복되는 파일, 컬렉션 null 체크 로직 존재 - CascadeType.ALL 적용
- put 메서드라 매번 전체 프로필 정보가 request에 전달돼서 문제가 생겼다. - 닉네임 중복 체크 시 자신의 닉네임인지도 확인하도록 수정
- isFollwer의 주체가 누군지 헷갈리는 것을 방지하기 위해 메서드 이름으로 의미를 나타냈다.
- 메세지가 100개까지만 로드됨 -> 모든 메세지가 보이도록 수정
- jwt 토큰에 닉네임 추가 - claims, details 등 수정 - 로그인 기능, jwtClaimsDto 등 리팩터링
### Exception 패키지 위치 변경
- 정적 변수에 숫자를 합산해서 저장했더니 값이 누적되는 버그 발생 - 멀티쓰레드 환경을 고려하여 인스턴스 변수로 변경
- approveStatus에 title값 추가
- title 값은 프론트에 내려주는 값
- 승인 요청 목록 보기 기능 구현
- 페이징 기본 10개씩
- 1차 정렬 기준 : 승인 대기 -> 보류 -> 트레이너 승인 -> 승인 거부 순으로 정렬
- enum 배치 순서대로 승인상태에 따라 1차 정렬
- Approval 입장에서 Certification 혹은 award 중 한쪽이 null 일 수 있다.
- UNION을 사용하고 정렬과 조회를 한번에 하기위해 네이티브 쿼리를 사용했다.
- 2차 정렬 기준 : ApprovalId 값 오름차순 (심사 요청 생성 일시 기준과 같다.)
- 관리자가 승인요청 목록을 보는 곳에서 상세보기하는 기능 구현 - 해당 승인요청이 존재하지 않을때 예외처리
- patch로 승인 상태를 변경하고 사유를 저장하는 기능 구현 - Oauth2JwtUserDetails에 잘못된 email 반환값 수정
- dto 모두 response로 변경 - @DaTa 제거 - 약간의 코드 정리 및 순서 조정
- chat 관련된 exception 패키지가 exception의 하위 패키지로 들어가있지 않아 수정함
- JwtUtils 수정
- getUid의 변수명이 올바르지 않아 getEmail로 수정
- Entity 수정
- ChatMessage : sender와 imageUrl을 없애고, user값으로 등록
- ChatRoom
- ChatRoom 안에서 chatRoom의 redisRoomId UUID값으로 생성
- ChatRoom Entity 안에 들어 있던 User 정보 삭제
- Controller 수정
- ChatMessageController : @Header에 토큰값을 받아와서 user 정보 받기
- ChatRoomController
- @ReqeustParm으로 받았던 userNickname값이 불필요하여 삭제
- @PathVariable 변수명을 좀 roomId가 아닌, redisRoomId로 명확하게 변경
- Service 수정
- ChatMessageService, ChatRoomService : userNickname user를 찾는 부분들 전부 수정
todo
- 현재 messageLoad 시 DB에서 모든 값을 가져오는 것을 프론트와 협의되어 있음.
추후 redis에서 채팅을 가져오고, 이에 대한 페이징 처리가 필요하게 된다면 주석처리 해놓은 부분을 사용할 예정
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.
No description provided.