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

✨ [Feat] 연관 레크레이션 목록 API구현 및 예외처리 추가 #53

Merged
merged 11 commits into from
Jan 31, 2024

Conversation

88dldl
Copy link
Collaborator

@88dldl 88dldl commented Jan 25, 2024

📍 PR 타입 (하나 이상 선택)

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

❗️ 관련 이슈 링크

Close #38

📌 개요

  • 연관 레크레이션 목록 API구현했습니다.

🔁 변경 사항

  • Elastic search로 하다가는 시간이 오래 걸릴 것 같아 Querydsl 사용했습니다 !
    -> 좀 더 나은 로직이 있다면 알려주세요!!!

  • 이전에 구현한 레크레이션 상세설명 부분에 예외처리 추가했습니다. 134ea2e

📸 스크린샷

👀 기타 더 이야기해볼 점

✅ 체크 리스트

  • PR 템플릿에 맞추어 작성했어요.
  • 변경 내용에 대한 테스트를 진행했어요.
  • 프로그램이 정상적으로 동작해요.
  • PR에 적절한 라벨을 선택했어요.
  • 불필요한 코드는 삭제했어요.

@88dldl 88dldl added ✨ feature New feature or request ♻️ refactor code refactoring labels Jan 25, 2024
@88dldl 88dldl self-assigned this Jan 25, 2024
@88dldl 88dldl linked an issue Jan 25, 2024 that may be closed by this pull request
2 tasks
@88dldl 88dldl changed the title ✨ [Feat] 연관레크레이션 목록 API구현 ✨ [Feat] 연관레크레이션 목록 API구현 및 예외처리 추가 Jan 25, 2024
@88dldl 88dldl changed the title ✨ [Feat] 연관레크레이션 목록 API구현 및 예외처리 추가 ✨ [Feat] 연관 레크레이션 목록 API구현 및 예외처리 추가 Jan 25, 2024
Copy link
Collaborator

@eomgerm eomgerm left a comment

Choose a reason for hiding this comment

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

이거 제가 말씀드린 거랑 좀 다르게 하셨는데... 혹시 그냥 제가 수정해서 커밋해도 될까...요...?

@88dldl
Copy link
Collaborator Author

88dldl commented Jan 26, 2024

이거 제가 말씀드린 거랑 좀 다르게 하셨는데... 혹시 그냥 제가 수정해서 커밋해도 될까...요...?

아 넵!! 그래주시면 감사하겠습니다 !

# Conflicts:
#	src/main/java/com/avab/avab/security/SecurityConfig.java
# Conflicts:
#	src/main/java/com/avab/avab/controller/UserController.java
#	src/main/java/com/avab/avab/repository/RecreationCustomRepositoryImpl.java
@eomgerm eomgerm merged commit ba56d18 into master Jan 31, 2024
@eomgerm eomgerm deleted the feat/38-related-list branch January 31, 2024 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor code refactoring ✨ feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ [Feat] 연관 레크레이션 목록 API 구현
3 participants