-
Notifications
You must be signed in to change notification settings - Fork 1
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] 봉사 지원 조회 관련 리팩토링 #337
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다!
파사드 서비스 잘 봤습니다~
사소한 피드백을 드리자면, Map 형태의 변수명에 Map이라는 표현만 사용되다 보니, 해당 자료가 무엇을 의미하는지 조금 이해하기 어려운 부분이 있었던 것 같습니다.
만약 AToB나 AByB 같은 네이밍 패턴을 활용한다면, 자료의 의도를 더 명확히 드러낼 수 있을 것 같아요!
ce7df37
to
bd31c77
Compare
bd31c77
to
f2c9506
Compare
Quality Gate passedIssues Measures |
resolved :
📌 과제 설명
봉사 지원 조회 관련 리팩토링
👩💻 요구 사항과 구현 내용
✅ PR 포인트 & 궁금한 점
이쪽부분 자세히 봐주시면 감사하겠습니다.