Skip to content
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

[LIME-33] 아이템 폴더 목록 조회 추가 및 기존 기능 수정 #6

Merged
merged 17 commits into from
Jan 20, 2024
Merged

[LIME-33] 아이템 폴더 목록 조회 추가 및 기존 기능 수정 #6

merged 17 commits into from
Jan 20, 2024

Conversation

Curry4182
Copy link
Contributor

@Curry4182 Curry4182 commented Jan 15, 2024 •
edited
Loading

📌 PR 종류

어떤 종류의 PR인지 아래 항목 중에 체크 해주세요.

  • 🐛 버그 수정
  • ✨ 기능 추가
  • ✅ 테스트 추가
  • 🎨 코드 스타일 변경 (formatting, local variables)
  • 🔨 리팩토링 (기능 변경 X)
  • 💚 빌드 관련 수정
  • 📝 문서 내용 수정
  • ✅ 기존 테스트 수정

📌 어떤 기능이 추가 되었나요?

  1. 멤버 아이템 폴더를 커서 기반으로 조회 할 수 있도록 기능 추가 하였습니다. 😎😊
  1. 아이템 엔티티에 폴더 아이디 추가로 인해 기존 아이템 관련 테스트를 수정하였습니다! 😊😆
  1. 아이템 엔티티에 folder id 추가로 인한 기존 api를 수정하였습니다! 😊😀 67641db 8881d58

Issue Number

LIME-33

기능 설명

😀 폴더 목록 조회

  • 기존 커서 방식을 이용하여 폴더 목록 조회를 할 수 있도록 기능 추가하였습니다!

📌 기존에 있던 기능에 영향을 주나요?

  • 네
  • 아니요

@Curry4182 Curry4182 self-assigned this Jan 15, 2024
@Curry4182 Curry4182 added the enhancement New feature or request label Jan 15, 2024
@Curry4182 Curry4182 marked this pull request as draft January 15, 2024 20:08
Yiseull
Yiseull approved these changes Jan 15, 2024
View reviewed changes
Copy link
Member

@Yiseull Yiseull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

병곤님 고생하셨습니다~! 궁금한 내용 하나 있어서 간단한 코멘트 남겼습니다~!!

...n/src/main/java/com/programmers/lime/domains/item/repository/MemberItemFolderRepository.java Outdated
Comment on lines 12 to 22
@Query(
"""
SELECT COUNT(mi)
FROM MemberItemFolder mi
WHERE mi.hobby = :hobby AND mi.memberId = :memberId
"""
)
int countByHobbyAndMemberId(
final Hobby hobby,
final Long memberId
);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p1;
JPA를 사용하지 않고 직접 쿼리를 작성하신 이유가 있나요??

Copy link
Contributor Author

@Curry4182 Curry4182 Jan 17, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인해보니 굳이 추가하지 않아도 되서 삭제했습니당 🙂 0efbed5

@Curry4182 Curry4182 marked this pull request as ready for review January 16, 2024 12:01
HandmadeCloud
HandmadeCloud approved these changes Jan 17, 2024
View reviewed changes
lime-api/src/main/java/com/programmers/lime/domains/item/api/ItemController.java Outdated
Comment on lines 130 to 142
@Operation(summary = "나의 아이템 목록 조회", description = "나의 아이템 목록을 조회 합니다.")
@GetMapping("/myitems")
@GetMapping("/myitems/folders/{folderId}")
public ResponseEntity<MemberItemGetByCursorResponse> getMemberItemsByCursor(
@PathVariable final Long folderId,
@RequestParam(required = false) final String hobbyName,
@ModelAttribute("request") @Valid final CursorRequest request
) {
Hobby hobby = Hobby.from(hobbyName);
MemberItemGetServiceResponse serviceResponse = itemService.getMemberItemsByCursor(
hobby,
folderId,
request.toParameters()
);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

폴더 상세조회로 swagger 설명도 바꿔보면 좋을 것 같아요~

@HandmadeCloud
Copy link
Member

질문

  • 아이템은 항상 폴더에 들어있어야하는 건가요?? 요구사항을 그렇게 생각하셨던건지 궁금합니당

고생하셨습니다~

@Curry4182
Copy link
Contributor Author

아이템이 항상 폴더에 있어야 한다는 요구사항 스크럼 시간 때 한번 더 의논해보면 좋을 것 같습니다!

@Curry4182 Curry4182 merged commit a8957bc into main Jan 20, 2024
HandmadeCloud added a commit that referenced this pull request Jan 22, 2024
@HandmadeCloud
Squashed commit of the following:
39b91d3 
commit fa63c9e
Author: ByeongGon Kang <[email protected]>
Date:   Mon Jan 22 17:19:55 2024 +0900

    hotfix: access token 반환 할 경우 '='이 없어서 프론트에 전달 되지 않는 버그 수정

commit d0d4581
Author: ByeongGon Kang <[email protected]>
Date:   Mon Jan 22 17:17:13 2024 +0900

    hotfix: access token 반환 경로 수정

commit 56ebc9f
Merge: b70c01d c7b5c61
Author: Yiseul Park <[email protected]>
Date:   Mon Jan 22 16:24:13 2024 +0900

    Merge pull request #17 from uju-in/NO-JIRA-fix-nickname-korean

    [NO-JIRA] 닉네임 중복 시 예외 대신 bool 타입으로 반환하도록 변경

commit c7b5c61
Author: Yiseul Park <[email protected]>
Date:   Mon Jan 22 16:14:25 2024 +0900

    test: 변경사항에 맞게 테스트 수정

commit 44f9005
Author: Yiseul Park <[email protected]>
Date:   Mon Jan 22 16:12:33 2024 +0900

    feat: 닉네임 중복값 bool 타입으로 반환

commit b70c01d
Merge: f0bd865 9bbb3b1
Author: Yiseul Park <[email protected]>
Date:   Mon Jan 22 15:47:52 2024 +0900

    Merge pull request #16 from uju-in/NO-JIRA-fix-nickname-korean

    [NO-JIRA] 프로필 업데이트, 닉네임 중복확인 request dto 유효성 검증 추가

commit 9bbb3b1
Author: Yiseul Park <[email protected]>
Date:   Mon Jan 22 15:17:24 2024 +0900

    feat: 닉네임 최소 길이 1로 변경

commit 0e7e0df
Author: Yiseul Park <[email protected]>
Date:   Mon Jan 22 15:11:42 2024 +0900

    fix: 프로필 업데이트, 닉네임 중복확인 request dto 유효성 검증 추가

commit f0bd865
Merge: 8e3f967 ed11b83
Author: JoJaeHyeon <[email protected]>
Date:   Sun Jan 21 00:36:20 2024 +0900

    Merge pull request #13 from uju-in/kakao-login

    [LIME-1] 일부 버그 수정. redirect 에 토큰 정보 쿠키로 전달

commit 8e3f967
Author: Curry4182 <[email protected]>
Date:   Sat Jan 20 23:48:49 2024 +0900

    hotfix: 컨트롤러에 중복되는 메서드 삭제

commit f24e122
Merge: a8957bc 9d7bb81
Author: ByeongGon Kang <[email protected]>
Date:   Sat Jan 20 23:18:37 2024 +0900

    Merge pull request #14 from uju-in/LIME-33-BE-get-folders-by-cursor

    [LIME-39] 멤버 아이템 폴더 생성, 삭제, 수정 기능 추가를 main에 반영하기 위한 PR

commit 9d7bb81
Author: Curry4182 <[email protected]>
Date:   Sat Jan 20 23:13:56 2024 +0900

    refactor: 사용하지 않는 변수 삭제

commit ee261b9
Merge: 1d29d62 bb1642c
Author: ByeongGon Kang <[email protected]>
Date:   Sat Jan 20 22:35:41 2024 +0900

    Merge pull request #10 from uju-in/LIME-39-folders-create-modify-remove

    [LIME-39] 멤버 아이템 폴더 생성, 삭제, 수정 기능 추가

commit bb1642c
Merge: cb24922 1d29d62
Author: ByeongGon Kang <[email protected]>
Date:   Sat Jan 20 17:39:05 2024 +0900

    Merge branch 'LIME-33-BE-get-folders-by-cursor' into LIME-39-folders-create-modify-remove

commit a8957bc
Merge: f333e1c 1d29d62
Author: ByeongGon Kang <[email protected]>
Date:   Sat Jan 20 16:47:44 2024 +0900

    Merge pull request #6 from uju-in/LIME-33-BE-get-folders-by-cursor

    [LIME-33] 아이템 폴더 목록 조회 추가 및 기존 기능 수정

commit f333e1c
Merge: 051bb66 2f6bebb
Author: Yiseul Park <[email protected]>
Date:   Fri Jan 19 14:18:56 2024 +0900

    Merge pull request #12 from uju-in/NO-JIRA-apply-changed-profile-update-requirements

    [N�O-JIRA] 변경된 요구사항에 맞게 프로필 정보 업데이트 기능 수정

commit 2f6bebb
Author: Yiseul Park <[email protected]>
Date:   Fri Jan 19 08:47:20 2024 +0900

    test: 프로필 업데이트 단위테스트 수정

commit df24605
Author: Yiseul Park <[email protected]>
Date:   Fri Jan 19 08:37:24 2024 +0900

    feat: 취미 경력 연차로 받기

commit 051bb66
Merge: c1a7eff 25c501b
Author: JoJaeHyeon <[email protected]>
Date:   Thu Jan 18 17:25:54 2024 +0900

    Merge pull request #11 from uju-in/kakao-login

    [LIME-1] Oauth2UserService 위치 변경

commit cb24922
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 21:46:34 2024 +0900

    feat: 나의 아이템 폴더 이름 최소, 최대 글자 설정

commit 1d29d62
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 15:17:00 2024 +0900

    docs: 스웨거 상세 설명 수정

commit 0efbed5
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 15:09:17 2024 +0900

    refactor: 작성하지 않아도 되는 sql 쿼리 삭제

commit c1a7eff
Merge: fe26ab8 4d6872a
Author: JoJaeHyeon <[email protected]>
Date:   Wed Jan 17 14:18:15 2024 +0900

    Merge pull request #7 from uju-in/kakao-login

    [LIME-1] Kakao 소셜로그인 구현

commit bb4057a
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 13:38:29 2024 +0900

    feat: 나의 아이템에서 제거할 때 폴더 별로 제거할 수 있도록 기능 추가

commit a27fce7
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 13:02:59 2024 +0900

    feat: 나의 아이템 폴더 조회할 경우 자신의 폴더일 때만 조회할 수 있도록 기능 추가

commit 035657e
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 13:02:15 2024 +0900

    remove: 나의 아이템 조회에서 폴더 추가로 필수적이지 않는 값 제거

commit d9ad57f
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 12:26:53 2024 +0900

    refactor: api 모듈에 있는 검증 클래스를 domain 모듈로 이동

commit 0e8a1ca
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 12:12:07 2024 +0900

    refactor: 아이템을 담을 때 아이템 랭킹 업데이트 하는 로직 리팩토링

commit 79d0156
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 12:09:17 2024 +0900

    feat: member item과 관련된 error 코드 추가

commit 34c4c49
Author: Curry4182 <[email protected]>
Date:   Wed Jan 17 12:08:25 2024 +0900

    rename: 나의 아이템인 경우 memberItem으로 변경, api에서 생성하는 경우는 add에서 create로 변경, 도메인에서 생성하는 경우 add에서 append로 변경

commit dcc12b3
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 21:38:03 2024 +0900

    refactor: Member Item Folder와 관련된 기능을 ItemService에서 MemberItemFolderService로 이동

commit 017fe57
Merge: e1de267 19b8114
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 21:19:44 2024 +0900

    Merge branch 'LIME-33-BE-get-folders-by-cursor' into LIME-39-folders-create-modify-remove

    # Conflicts:
    #	lime-api/src/main/java/com/programmers/lime/domains/item/application/ItemService.java

commit 19b8114
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 21:00:31 2024 +0900

    test: 멤버 아이템 폴더 id 변경으로 인한 테스트 수정

commit 1edb16f
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 20:59:43 2024 +0900

    fixture: 멤버 아이템 폴더 추가 테스트를 위한 fixture 추가

commit fcf30c4
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 20:57:39 2024 +0900

    feat: 멤버 아이템과 관련된 로직에 folder id 추가

commit ce39907
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 15:38:05 2024 +0900

    remove: 아이템 관련 기능에서 folder id 가 있는 기능 삭제

commit e2c7d6e
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 15:34:02 2024 +0900

    feat: folder id를 기존 Item 엔티티에서 MemberItem으로 변경

commit 90e8831
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 15:19:47 2024 +0900

    remove: 아이템 등록할 경우 나의 아이템 추가 기능 삭제

commit e1de267
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 04:58:41 2024 +0900

    feat: 멤버 아이템 폴더 삭제 api 추가

commit 297112c
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 03:50:56 2024 +0900

    feat: 멤버 아이템 폴더 수정 api 추가

commit bde217a
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 03:25:06 2024 +0900

    feat: 멤버 아이템 폴더 도메인 read 기능 추가

commit 13c5edb
Author: Curry4182 <[email protected]>
Date:   Tue Jan 16 01:48:15 2024 +0900

    feat: 아이템 폴더 생성 기능 추가

commit 2dff796
Author: Curry4182 <[email protected]>
Date:   Mon Jan 15 17:15:23 2024 +0900

    test: hobby 변경으로 인한 기존 테스트 변경

commit 8881d58
Author: Curry4182 <[email protected]>
Date:   Mon Jan 15 17:12:29 2024 +0900

    feat: 아이템 엔티티에 폴더 id 추가로 인한 기존 api 수정

commit 67641db
Author: Curry4182 <[email protected]>
Date:   Mon Jan 15 17:12:29 2024 +0900

    feat: 아이템 엔티티에 폴더 id 추가로 인한 기존 api 수정

commit 70c0043
Author: Curry4182 <[email protected]>
Date:   Mon Jan 15 17:08:45 2024 +0900

    feat: 아이템 엔티티에 폴더 id 추가

commit b2a3de8
Author: Curry4182 <[email protected]>
Date:   Mon Jan 15 16:50:45 2024 +0900

    feat: 도메인 모듈에 멤버 아이템 폴더 조회 기능 추가

commit 7c1b9e3
Author: Curry4182 <[email protected]>
Date:   Mon Jan 15 16:48:09 2024 +0900

    feat: 멤버 아이템 폴더 도메인 추가

commit 1483acf
Author: Curry4182 <[email protected]>
Date:   Mon Jan 15 16:46:14 2024 +0900

    test: 아이템 관련 도메인 테스트 할 경우 폴더 아이디 추가

commit 86c3aa6
Author: Curry4182 <[email protected]>
Date:   Mon Jan 15 16:45:06 2024 +0900

    test: 아이템 빌더 폴더 아이디 추가

commit fbd51c8
Author: Curry4182 <[email protected]>
Date:   Mon Jan 15 16:43:47 2024 +0900

    feat: 멤버 아이템 폴더 커서 조회 기능 추가
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Yiseull Yiseull Yiseull approved these changes

@HandmadeCloud HandmadeCloud HandmadeCloud approved these changes

Assignees

@Curry4182 Curry4182

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Curry4182 @HandmadeCloud @Yiseull