-
Notifications
You must be signed in to change notification settings - Fork 8
지원서 지원 학기 필드 추가 #71
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
지원서 지원 학기 필드 추가 #71
Conversation
|
추가 작업내용입니다.
주의사항 지원 프로세스가 변경되었습니다. 성적 제출
지망대학 제출
추후 디벨롭 방안
확실히 테이블을 분리해야 flow가 자연스러워질 수 있을 것 같습니다. |
wibaek
left a comment
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.
기존 성적있을시에 지망대학 제출하면 성적 가져오는 로직 좋은 것 같습니다.
테스트 문제없으면 머지해도 될 것 같습니다. 수고하셨습니다 👍
필요하시면 영서님에게 리뷰 요청 드려도 될 것 같습니다.
| VerifyStatus verifyStatus = applicationRepository.getApplicationBySiteUser(siteUser).getVerifyStatus(); | ||
| if (verifyStatus != VerifyStatus.APPROVED) { | ||
| throw new CustomException(APPLICATION_NOT_APPROVED); | ||
| } |
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.
validateSiteUserCanViewApplicants() 에서 applicationRepository.getApplicationBySiteUserAndTerm(siteUser, term) 이런식으로 변경해서 현재 term에 인증되었는지 확인해야할 것 같습니다.
아니면 2024-2 인증자가 2025-1 성적 지원도 확인할 수 있는 문제가 발생할 수 있습니다.
| Gpa gpa = scoreRequest.toGpa(); | ||
| LanguageTest languageTest = scoreRequest.toLanguageTest(); | ||
|
|
||
| applicationRepository.findBySiteUser_Email(email) |
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.
applicationRepository.findByTermAndSiteUser_Email(term, email)로 체크해야 2024-2 기록을 유지한 채로 2025-1 기록을 새롭게 등록할 수 있을 것 같습니다
현재 상황에서는 2024-2 기록을 등록한 채로 새롭게 학점과 영어 성적만을 제출한 유저가 있을 때, 현재 성적 공유 조회 로직 상으로는 자신의 지원 대학을 2025-1 대학으로 업데이트 하지 않고도 2025-1 성적 공유 페이지를 조회할 수 있게 됩니다
마찬가지로 79번째 줄 submitUniversityChoice()에서도 applicationRepository.findByTermAndSiteUser_Email(term, email)로 체크해야할 것 같습니다
This reverts commit bf27f61.
관련 이슈
작업 내용
application에 term 추가
지원서 추가시 현재 term으로 삽입하게 변경
성적 공유 목록 확인시 현재 term의 목록만 조회하게 변경
테스트코드 작성하였습니다.
특이 사항
Application 필드 추가로 배포시 운영DB에 필드를 반영해줘야 합니다.
리뷰 요구사항 (선택)