This repository has been archived by the owner on Aug 13, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Model 관련 * BlockUserListRequest 클래스 추가 - 현재 사용자 아이디, 차단 목록 페이지 위치, 그리고 한 페이지에 출력할 아이디 개수의 제한 숫자를 멤버 변수로 갖는 인스턴스를 생성할 클래스 추가 Controller 계층 관련 * BlockUserController - 현재 로그인 중인 사용자의 차단 사용자 목록을 불러오는 요청을 받을 `getBlockUserList` 메소드 추가 Service 계층 관련 * BlockUserService - 현재 사용자의 차단 목록을 불러오는 비즈니스 로직을 처리할 `getBlockUserList` 메소드 추가 - `@Transactional(readOnly=true)를 붙여서 Slave DB에서 내용을 읽어오도록 설정 Mapper 계층 관련 * BlockUserMapper - DB에서 차단 사용자 목록을 불러 올 `getBlockUserList` 메소드 추가 * BlockUserMapper.xml - 위의 내용을 처리할 쿼리문 추가 Util 관련 * Pagination - 차단 목록에 대한 max per page 멤버변수 추가 테스트 관련 - 포스트맨을 사용해 위의 로직 테스트
1 task
브랜치를 착각해서 |
f-lab-dev
reviewed
Mar 16, 2021
private final int maxPageBlockUser; | ||
|
||
public Pagination(@Value("${friend.max.page:30}") int maxPageFriend, | ||
@Value("${block_user.max.page:30}") int maxPageBlockUser) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이쪽 부분은 이전 리뷰에 남겨드린것처럼 페이징 계산식까지 아예 여기로 가져와서 페이징 계산용 클래스로 만들어줘도 좋겠네요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 머지한다음 계산식 만들었던거 적용해서 계산용 클래스로 만들어줬습니다!
f-lab-dev
approved these changes
Mar 18, 2021
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
📖 Model 관련
📑 BlockUserListRequest 클래스 추가
📖 Controller 계층 관련
📑 BlockUserController
getBlockUserList
메소드 추가📖 Service 계층 관련
📑 BlockUserService
현재 사용자의 차단 목록을 불러오는 비즈니스 로직을 처리할
getBlockUserList
메소드 추가`@Transactional(readOnly=true)를 붙여서 Slave DB에서 내용을 읽어오도록 설정
📖 Mapper 계층 관련
📑 BlockUserMapper
getBlockUserList
메소드 추가📑 BlockUserMapper.xml
📖 Util 관련
📑 Pagination
📖 테스트 관련