RAC-257 feat : 결제 기능 추가 및 리팩토링" #89
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ay-eonii
requested changes
Jan 7, 2024
Comment on lines
+54
to
+59
| List<SalaryInfo> salaryInfos = seniors.stream() | ||
| .map(senior -> { | ||
| Salary salary = salaryGetService.bySenior(senior.senior()); | ||
| return getSalaryInfo(senior.senior(), salary); | ||
| }) | ||
| .toList(); |
There was a problem hiding this comment.
public Salary bySenior(Senior senior) {
LocalDate salaryDate = SalaryUtil.getSalaryDate();
return salaryRepository.findBySeniorAndSalaryDate(senior, salaryDate).orElseThrow(SalaryNotFoundException::new);
}정산정보를 현재의 salaryDate 를 기준으로 가져오던데, 이러면 과거의 정산 정보는 못 보지 않나요?
그리고 완료된 항목만 노출해야 할 것 같아요!
There was a problem hiding this comment.
@GetMapping("/salary")
@Operation(summary = "[관리자] 정산 목록 조회", description = "한 달 기준으로 정산 목록을 조회합니다. 기준일은 [11일 ~ 내월 10일]입니다.")
public ResponseDto<SalaryManageResponse> getSalaries(@RequestParam(required = false) Integer page,
@RequestParam(required = false) String search) {
SalaryManageResponse salaries = salaryManageUseCase.getSalaries(page, search);
return ResponseDto.create(ADMIN_FIND.getCode(), GET_LIST.getMessage(), salaries);
}컨트롤러에 이렇게 나와있어서 맞춰서 했었는데 저도 이 부분에 대해서는 논의를 해보려고 했어요!
일단 이 부분은 방식에 대해서 추가로 논의하는게 어떨까요? 관리자 부분은 후순위이다 보니
There was a problem hiding this comment.
아하 제가 이후에 로직을 수정하고 설명은 수정을 안 했네요 ㅜㅜ
There was a problem hiding this comment.
멘토링을 취소 또는 거절하는 경우에 payment 변경 로직이 필요할 것 같습니닷..
There was a problem hiding this comment.
이부분은 환불 로직을 추가하면서 하려고 합니다!
ay-eonii
approved these changes
Jan 7, 2024
Comment on lines
+54
to
+59
| List<SalaryInfo> salaryInfos = seniors.stream() | ||
| .map(senior -> { | ||
| Salary salary = salaryGetService.bySenior(senior.senior()); | ||
| return getSalaryInfo(senior.senior(), salary); | ||
| }) | ||
| .toList(); |
There was a problem hiding this comment.
아하 제가 이후에 로직을 수정하고 설명은 수정을 안 했네요 ㅜㅜ
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🦝 PR 요약
기본 결제 기능 추가 및 리팩토링
✨ PR 상세 내용
결제 관련 추가적인 부분은 다음 브랜치에서 작업하도록 하겠습니다!
Payment테이블에 대한 수정이 생기면서 여러 테이블에 모두 영향을 끼치게 되었습니다..
변경된 부분
참고로, Salary에 accountNumber, accountHolder, bank가 추가되었습니다. 그리고 totalAmount가 추가되어 총 정산 금액이 저장되고 있습니다.
Mentoring에 Pay가 삭제되었습니다. (필요가 없다고 생각했습니다)
🚨 주의 사항
Payment는 아직 더 다듬을 생각입니다!
많은 부분에서 수정이 생겼으니 기존의 로직과 충돌이 발생하지는 않는지 세세한 확인이 필요합니다!
또한 많은 로직이 변경되어 해당 부분에 대한 확인 및 물어보시면 되겠습니다!
테스트는 모두 진행을 하였지만 문제가 있는 부분이 있다면 빠른 피드백 바랍니다!
실수로 머지했다가 revert하고 다시 revert하는 pr을 올립니다....ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ
✅ 체크 리스트