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.

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

프로젝트 상세 조회 QueryDSL 적용 및 리팩토링 #70

Merged
merged 19 commits into from
Feb 27, 2024

Conversation

yenzip
Copy link
Contributor

@yenzip yenzip commented Feb 26, 2024

🎫 관련 이슈

Resolves #57

✅ 구현 내용

  • 프로젝트 상세 조회 Service 코드 리팩토링
  • 프로젝트 상세 조회 - Member 조회 QueryDSL 적용
  • 프로젝트 상세 조회 테스트 코드 작성

💬 코멘트

QueryDSL 관련 블로깅을 작성했습니다..! 사이드픽에서 QueryDSL 적용하기까지 여정
확인해보시고 피드백 주실 것이 있다면 언제든 편하게 해주세요!
@Sehee-Lee-01 프로젝트 생성 관련 테스트 할 때, 제가 만든 메서드(createXXX)를 편하신 대로 변경해서 사용하셔도 됩니다..!

@yenzip yenzip added ✏️ 기록혀야쥬 문서 기록 🚀 더 만들어봤슈 기능 구현 ⚙️ 세팅했슈 환경 세팅 🛠 리팩토링했슈 리팩토링 ✅ 테스트혀봤슈 빌드&테스트가 필요할 때 태그 labels Feb 26, 2024
@yenzip yenzip added this to the 🚀 2차 스프린트 milestone Feb 26, 2024
@yenzip yenzip self-assigned this Feb 26, 2024
@Sehee-Lee-01
Copy link
Member

혹시 의진이 커밋이랑 합쳐진 것 같은데 나중에 확인 한 번만 부탁드립니다!

@yenzip
Copy link
Contributor Author

yenzip commented Feb 27, 2024

혹시 의진이 커밋이랑 합쳐진 것 같은데 나중에 확인 한 번만 부탁드립니다!

엇 커밋 기록이 합쳐졌네용..! 해결해보겠습니다..!

Copy link
Member

@Sehee-Lee-01 Sehee-Lee-01 left a comment

Choose a reason for hiding this comment

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

예림님,,아침까지 수고많으셨습니다,, 아니 너무너무너무 수고많으셨습니다,.,,
추가로 현재 브랜치 제 작업 브랜치에서 머지해서 작업하고 잇어요! 예림님 브랜치 머지되고나서 제꺼 머지해야할겁니다! 참고해주셔용!!

커밋 내용 겹쳐진 것은 알장딱갈센으로 제가 골라서 봤습니다! 보는데 크게 불편한 건 없어서 머지하면 문제는 없을 것 같어유! ㅠㅠ 너무 수고 많으셨성요ㅠㅠㅠㅠ


public final NumberPath<Long> ownerId = createNumber("ownerId", Long.class);

public final DateTimePath<java.time.LocalDateTime> startDate = createDateTime("startDate", java.time.LocalDateTime.class);
Copy link
Member

Choose a reason for hiding this comment

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

요 부분은 예림님 지금 브랜치 제 작업 브랜치에 머지해서 보니 YearMonth로 되어있드라구요!
혹시 몰라서 코멘트만 남겨놓을게유!
저도 YearMonth인 상태로 작업했어요!!

@@ -42,6 +43,7 @@ public static ProjectResponse from(Project project, List<OverviewImageSummary> o
.techStacks(techStacks)
.startDate(YearMonth.from(project.getStartDate()))
.endDate(YearMonth.from(project.getEndDate()))
.ownerId(project.getOwnerId())
Copy link
Member

Choose a reason for hiding this comment

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

저도 추가해서 제 브랜치에서 conflict 해결했습니다용!! 여기서두 써주셔서 감사합니당!!👍

import sixgaezzang.sidepeek.users.dto.response.UserSummary;

@Repository
public class MemberRepositoryCustomImpl implements MemberRepositoryCustom {
Copy link
Member

Choose a reason for hiding this comment

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

👍👍👍
쿼리dsl 잘 배워갑니당!!!

Comment on lines +77 to +78
LocalDateTime startDate = LocalDateTime.now();
LocalDateTime endDate = startDate.plusMonths(3);
Copy link
Member

Choose a reason for hiding this comment

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

요 부분도 제가 YearMonth 미리 머지해서 수정햇습니다! 감사합니다ㅠㅠㅠㅠ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ 세팅했슈 환경 세팅 ✅ 테스트혀봤슈 빌드&테스트가 필요할 때 태그 ✏️ 기록혀야쥬 문서 기록 🚀 더 만들어봤슈 기능 구현 🛠 리팩토링했슈 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants