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] 응답형식 리팩토링 #88

Merged
merged 1 commit into from
Mar 10, 2024
Merged

[Refactor] 응답형식 리팩토링 #88

merged 1 commit into from
Mar 10, 2024

Conversation

choeun7
Copy link
Contributor

@choeun7 choeun7 commented Mar 4, 2024

✨ PR 유형

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

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

🛠️ 작업내용

Service 단에 무의미한 반복이 많아서, ApiResponse를 반환하는 방법으로
바꿨습니다. Benefit 파일만 우선 예시로 수정했습니다.

📋 추후 진행 상황

대면 회의에서 다른 파일들도 수정할 예정입니다.

📌 리뷰 포인트

따로 응답 파일을 더 만들지 않고, 이미 사용하던 ApiResponse를 이용해 Service 코드를 줄였습니다.
200 이 아닌 다른 코드가 필요하면 Controller에서 if/else 문으로 다루어야 할 것 같습니다.

✅ Checklist

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

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

Service 단에 무의미한 반복이 많아서, ApiResponse를 반환하는 방법으로
바꿨습니다. Benefit 파일만 우선 예시로 수정했습니다.
@choeun7 choeun7 added 🚶‍♀️초은 초은이 담당 ⚒️ refactor 리팩토링 labels Mar 4, 2024
@choeun7 choeun7 self-assigned this Mar 4, 2024
Copy link
Member

@arinming arinming left a comment

Choose a reason for hiding this comment

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

헙 ApiResponse가 훨씬 깔끔하네요 수고하셨습니당

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.

깔끔합니다! 고생하셨어요!

Copy link
Contributor

@Jeongmin39 Jeongmin39 left a comment

Choose a reason for hiding this comment

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

확인했습니다 고생하셨습니다!

@choeun7 choeun7 merged commit d68d231 into dev Mar 10, 2024
2 checks passed
@arinming arinming deleted the refactor/87 branch March 10, 2024 10:00
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.

4 participants