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

[#43] UseCase 응답 객체 이름 변경 #49

Merged
merged 1 commit into from
Oct 22, 2024
Merged

[#43] UseCase 응답 객체 이름 변경 #49

merged 1 commit into from
Oct 22, 2024

Conversation

Srltas
Copy link
Collaborator

@Srltas Srltas commented Oct 20, 2024

📌 Summary

UseCase에서 응답 객체의 이름을 Response에서 Result로 변경하여, 웹 계층에 한정된 느낌을 없애고 더 중립적인 이름을 사용했습니다. 이를 통해 응답 객체가 특정 기술에 의존하지 않도록 개선했습니다.

📝 Description

NeighborhoodVerificationResponse -> NeighborhoodVerificationResult

  • 컨트롤러, 서비스, 테스트 코드 등 모두 이 변경이 적용되었습니다.

✅ Checklist

  • 새로운 기능이나 수정된 기능에 대해 충분한 테스트를 작성했습니다.
  • 코딩 스타일 가이드를 준수했습니다.
  • 문서(주석, README 등)가 필요하다면 업데이트했습니다.

- 기존에 사용하던 Response 이름은 Web과 강하게 연결되어 있는 느낌의 이름
- Result로 변경하여 응답 객체가 특정 기술에 의존하지 않도록 변경
@Srltas Srltas added the refactor Code refactoring operations label Oct 20, 2024
@Srltas Srltas self-assigned this Oct 20, 2024
Copy link
Collaborator

@f-lab-bradley f-lab-bradley left a comment

Choose a reason for hiding this comment

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

각 계층에 분리된 이름을 적절히 사용 하셨네요 👍

@Srltas Srltas merged commit 44c6ea8 into main Oct 22, 2024
1 check passed
@Srltas Srltas deleted the issue_43 branch October 22, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Code refactoring operations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants