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

[BSVR-56] QueryDSL 설정 추가 #13

Merged
merged 8 commits into from
Jul 4, 2024
Merged

[BSVR-56] QueryDSL 설정 추가 #13

merged 8 commits into from
Jul 4, 2024

Conversation

EunjiShin
Copy link
Collaborator

📌 개요 (필수)

  • 복잡한 쿼리 & 동적 쿼리 컨트롤을 위해 queryDSL 의존성을 추가해요.

🔨 작업 사항 (필수)

  • queryDSL 의존성, 설정파일 추가
  • 샘플 코드 추가
  • editorconfig 인덴트 사이즈 조정
    • spotless에서 javaFormat.aosp를 사용 중이라 editorconfig랑 안맞는 이슈
      • 그냥 javaFormat은 인덴트2인데, aosp는 인덴트4
    • 티켓 파긴 작은 이슈라서 여기 PR에서 같이 고쳤어용

🌱 연관 내용 (선택)


💻 실행 화면

p6spy도 잘 동작하는 것 확인 완료!

스크린샷 2024-07-05 오전 12 13 30 스크린샷 2024-07-05 오전 12 13 44

@EunjiShin EunjiShin self-assigned this Jul 4, 2024
@EunjiShin EunjiShin added 🛠 Setting 개발 환경 세팅 🔨 Build 빌드 관련 (gradle 등) labels Jul 4, 2024
@github-actions github-actions bot added the size/M label Jul 4, 2024
Copy link

github-actions bot commented Jul 4, 2024

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 1a401eb. ± Comparison against base commit f088444.

@Configuration
public class QueryDslConfig {

@PersistenceContext private EntityManager entityManager;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

�EntityManager를 Spring에서 주입받아 사용하기 위해 @PersistenceContext를 이용한 코드

Copy link
Collaborator

@pminsung12 pminsung12 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!


@Override
public Member save(Member member) {
val memberEntity = memberJpaRepository.save(MemberEntity.from(member));
return memberEntity.toDomain();
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@pminsung12 pminsung12 merged commit 4dd7bfb into main Jul 4, 2024
4 checks passed
@pminsung12 pminsung12 deleted the feat/BSVR-56 branch July 4, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Build 빌드 관련 (gradle 등) 🛠 Setting 개발 환경 세팅 size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants