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

Review Controller 마이그레이션을 진행 #27

Merged
merged 35 commits into from
Sep 22, 2024
Merged

Review Controller 마이그레이션을 진행 #27

merged 35 commits into from
Sep 22, 2024

Conversation

sjhjack
Copy link
Contributor

@sjhjack sjhjack commented Sep 22, 2024

📑 개요

✅ PR 체크리스트


  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 💯 테스트는 잘 통과했나요?
  • 🏗️ 빌드는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?

🚀 상세 작업 내용


  • Review Controller 마이그레이션
     - ResponseEntity 변경
      - WyplResponseEntity 적용
     - @Authenticated 작성 및 적용
      - AuthMember 추가
      - AuthenticationArguementResolver 작성
      - WebConfig 설정
     - H2, embedded MongoDB 설정 추가

  • Review Service 파라미터 변경
     - Member Entity의 id -> member_id 변경
     - 쿼리문에 member_id 적용

📁 ETC


Copy link

📄 Image Server Jacoco Coverage Report

Overall Project 88.81% 🍏

There is no coverage information present for the Files changed

Copy link
Member

@KIMSEI1124 KIMSEI1124 left a comment

Choose a reason for hiding this comment

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

LGTM!

@sjhjack sjhjack merged commit b4cb908 into dev Sep 22, 2024
2 of 3 checks passed
@sjhjack sjhjack deleted the feat/#23 branch September 22, 2024 10:59
Copy link

📄 Image Server Jacoco Coverage Report

Overall Project 88.81% 🍏

There is no coverage information present for the Files changed

@sjhjack sjhjack mentioned this pull request Sep 22, 2024
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Review Controller 마이그레이션을 진행한다.
2 participants