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.

Sign up for GitHub

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

Jump to bottom

[IDLE-231] fix: 마이북 목록 조회 API 수정 #79

Merged

0112leesy merged 1 commit into master from bug/IDLE-231

Aug 22, 2023

Member

0112leesy commented Aug 22, 2023

🧑‍💻 작업 사항

버그 상황

앱 테스트 중, 다른 사용자의 프로필에서 마이북을 조회했을 때
방문한 사용자의 마이북 목록이 아니라
로그인한 사용자의 마이북 목록이 보여졌습니다.

원인

마이북 목록 조회 Controller에서 서비스 요청 DTO를 생성할 때
생성자의 매개변수 순서가 바뀌었기 때문입니다.

해결

userId 와 loginId의 순서를 바꾸어줬습니다.

🔗 링크

🐰 시급한 정도

🚨 긴급 : 선 어프루브 후 리뷰를 부탁드립니다. (리뷰는 추후 반영)

📖 참고 사항

테스트 코드는 요청 DTO가 any()로 모킹 되어 있었기에 수정 작업이 없습니다.


          [IDLE-231] fix: MyBookFindAllServiceRequest DTO 생성자 매개변수 순서 변경

ff602e7

0112leesy requested review from minnseong and DonggyuJin

August 22, 2023 11:43

DonggyuJin approved these changes

View reviewed changes

Collaborator

DonggyuJin left a comment

아주 빠른 수정 감사드립니다 👍👍

0112leesy merged commit 3105454 into master

2 checks passed

0112leesy deleted the bug/IDLE-231 branch

August 22, 2023 17:18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet