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

[Refactor] 전체 파일 응답 호출방법 수정 #92

Merged
merged 3 commits into from
Mar 22, 2024
Merged

Conversation

choeun7
Copy link
Contributor

@choeun7 choeun7 commented Mar 11, 2024

✨ PR 유형

어떤 변경 사항이 있나요??

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

🛠️ 작업내용

ApiResponse 파일이랑 @ApiResponse 태그랑 겹쳐서 ApiResponse -> CommonDto로 수정했습니다.
일단 파일 내에서 제네릭<?>랑 apiResponse 변수명 없는거 확인하긴 했는데, 혹시 발견하신다면 말씀해주세요

📋 추후 진행 상황

추가 회의를 통해 결정해야할 것 같습니다.
상제님의 Error 처리방법이 업데이트되면 예외처리 코드 추가해야할 것 같습니다.

📌 리뷰 포인트

자기 코드 이상하게 수정되어있거나, 예외처리 누락된 것 있는지 확인해주세요
일단 MemberService 제외하고 모든 응답이 true로 되어있는 걸로 기억해요

✅ Checklist

PR이 다음 요구 사항을 충족하는지 확인해주세요

  • 커밋 메시지 컨벤션에 맞게 작성했습니다
  • 유지-보수를 위해 주석 처리를 잘 작성했습니다

choeun7 added 3 commits March 10, 2024 19:27
ApiResponse -> CommonDto
swagger의 @ApiResponse와 겹쳐서 클래스명 수정했습니다.
@choeun7 choeun7 added 🚶‍♀️초은 초은이 담당 ⚒️ refactor 리팩토링 labels Mar 11, 2024
@choeun7 choeun7 self-assigned this Mar 11, 2024
Copy link
Contributor

@sanggae4133 sanggae4133 left a comment

Choose a reason for hiding this comment

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

lgtm

@choeun7 choeun7 merged commit ea12807 into dev Mar 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚒️ refactor 리팩토링 🚶‍♀️초은 초은이 담당
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants