Release:v1.0.0-beta.12 #57
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
- userPostId가 실제 존재하는지 검증 - UUID를 조회해 유저가 탈퇴했다면 댓글 인스턴스의 uuid를 null로 변경 feat: first amend - 리스트 원소들을 outdto로 매핑하던 로직을 컨트롤러에서 서비스 로직으로 옮김 - 주석을 좀더 상세하고 구체적으로 수정, 메서드 상단에 [목적]명시 - 닉네임 필드, 닉네임 조회 로직 추가 feat: second amend - impl findAllByUserPostId 메서드 주석 최적화(메서드 최상단에 목적을 명시) - 서비스레이어에서 nickname, uuid를 각각 쿼리 두번에 받던걸 한번으로 최적화
- 현재 유저 참여여부를 나타내는 변수명이라 writerActiveState 대신 userParticipatingIn이 더 직관적이라 판단 - 전자는 글 게시자 탈퇴 여부 변수로 더 적합하지만 이는 uuid null 처리로 로직이 바뀌었음
- 현재 유저 참여여부를 나타내는 변수명이라 writerActiveState 대신 userParticipatingIn이 더 직관적이라 판단 - 전자는 글 게시자 탈퇴 여부 변수로 더 적합하지만 이는 uuid null 처리로 로직이 바뀌었음
- 삭제는 indto만 있고 나머진 수정 관련
- 수정 로직은 각 엔티티 객체 가져와 tobuilder로 새 정보들을 박아서 다시 save 하는 방식 택 feat: first amend - api 호출자가 수정/삭제 대상 소분글 게시자와 동일한지 체크 - imageUrl 수정부분 삭제로직 제거 - userPost에서 수정에 영향받는 서버자체계산 필드값들 최신화 로직 추가 - post 수정부분 modelMapper로 최적화 feat: second amend - dto 개행라인 통일 - 컨트롤러 build 개행여부 통일 - BaseResponseStatus 수정/삭제 지우고 기존 403으로 통일
- 내가 작성한 소분글, 내가 참여중인 소분글, 내가 참여한 소분이 완료된 소분글 조회를 type별로 조회하는 API 개발 - queryDsl에서 공통적으로 사용되는 부분을 컴포넌트 & 서비스 클래스로 묶어서 구현
- 모니터링을 위해 모든 api 요청시 log를 남겨주는 interceptor를 구현 - 이를 위해 request를 캐싱한 CachingRequestWrapper 구현 및 관련된 의존성 추가
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.
ygyg[v1.0.0-beta.12]