Skip to content

Conversation

@ssosee
Copy link
Member

@ssosee ssosee commented Nov 2, 2025

제목 : [(Jira issue 번호)] 작업명

ex) [DP-123] pull request template 작성
(확인 후 지워주세요)

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요 (이미지 첨부 가능)

  • 픽픽픽 검색 API 개발
    • orm 을 사용하기에는 조금 복잡해서 mybtis 를 사용했습니다.
  • 가독성을 위해서 픽픽픽 V1, V2 문서를 분리했습니다.

🔗 참고할만한 자료(선택)

슬랙이나 피그마, Jira 등 참고 링크를 첨부해주세요

  • [레포 이름 #이슈번호](이슈 주소)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@ssosee ssosee requested a review from yu-so-young2 November 2, 2025 05:35
@ssosee ssosee self-assigned this Nov 2, 2025
@ssosee ssosee added documentation Improvements or additions to documentation feat 기능 추가 labels Nov 2, 2025
Copy link
Member

@yu-so-young2 yu-so-young2 left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다!

"/devdevdev/api/v1/test/**",
"/devdevdev/api/v1/token/**",
"/devdevdev/api/v1/picks/**",
"/devdevdev/api/v2/picks/**",
Copy link
Member

Choose a reason for hiding this comment

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

요거 제가 놓쳤었는데 챙겨주셔서 감사합니다!

public class PickMainSearchResponseV2 extends PickMainResponseV2 {
private final Double searchScore;

@Builder(builderMethodName = "searchBuilder")
Copy link
Member

Choose a reason for hiding this comment

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

(단순 궁금증) 빌더 메서드 이름을 지정하셨는데, 특별한 이유가 있을까요?

Copy link
Member Author

@ssosee ssosee Nov 5, 2025

Choose a reason for hiding this comment

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

이거 상속 받을때 부모가 @Builder 선언 되어 있으면, 자식은 @Builder 그대로 사용 못해서, 네이밍 해줘야한다고 해서 저렇게 했어요!

Copy link
Member

Choose a reason for hiding this comment

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

아하 넵 설명 감사합니다!!

Copy link
Member

Choose a reason for hiding this comment

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

검색 시에는 pick.status = APPROVAL 상태인지 확인하지 않는 것 같은데, 이후에
findPicksWithPickOptionWithMemberByIdIn() 에서 조회하기 때문으로 이해하면 될까욥?!

Copy link
Member Author

Choose a reason for hiding this comment

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

오 넣는게 좋을 것 같아요! 감사합니다.

@yu-so-young2 yu-so-young2 merged commit 11210ee into develop Nov 5, 2025
1 check passed
@yu-so-young2 yu-so-young2 deleted the DP-560 branch November 5, 2025 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation feat 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants