[FEAT] 사용자의 네일 세트 보관함 구현 #34
Open
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
onehowon
requested changes
Mar 6, 2025
|
|
||
| public interface NailSetRepository extends ReactiveCrudRepository<NailSet, Integer> { | ||
| Flux<NailSet> findAllBy(Pageable page); | ||
|
|
||
| Flux<NailSet> findAllByUploadedBy(Integer uploadedUserId, Pageable page); | ||
|
|
There was a problem hiding this comment.
Flux<NailSet> findAllByUploadedBy(Integer uploadedUserId, Pageable page);
- 이 부분은 Integer로 업로드한 유저 아이디를 찾아오는데
Mono<NailSet> findByUploadedByAndNailGroupId(int uploadedUserId, int groupId);
- 이 부분은 int로 받아오는 이유가 있을까요? 가능하면 한 타입으로 일관하는게 좋을 것 같습니다.
There was a problem hiding this comment.
Integer 는 nullable 이지만 primitive type 인 int 는 NonNull이라서 이렇게 짰던 것 같네요. 양쪽 다 primitive type 인 int 로 수정하겠습니다.
| .uploadedBy(userId) | ||
| .build())) | ||
| ); | ||
| } |
There was a problem hiding this comment.
public Mono<NailSet> cloneNailSetForUser(int userId, int setId)
해당 메서드에서 다중 요청이 들어올 경우도 고려해야할까요? userId와 nailGroupId 조합에 대해 중복 보관이 발생할 가능성이 있을 것 같아서요
만약 그렇다면 데이터베이스 레벨에서 Unique Constraint가 있으면 좋을 것 같은데 의견 부탁드리겠습니다!
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.
🔖 관련 티켓
SN-58 (Jira 링크)
📌 작업 개요
네일 세트의 사용자 보관함 기능 구현입니다
✅ 작업 항목
📝 추가 설명
사용자가 네일 세트 보관을 여러 번 요청할 경우에, 이미 해당 네일 세트를 보관한 경우에는 기존에 보관했던 정보를 그대로 반환하며 따로 오류를 내주거나 다른 상태를 반환하진 않습니다.