issues
search
hwarrk
/
hwarrk-back
0
stars
1
forks
source link
이전 merged PR에 대한 코드 통합 및 리팩토링
#21
Closed
lsh2613
closed
1 week ago
lsh2613
commented
1 week ago
⭐ Summary
이전 merged PR에 대한 코드 통합 및 리팩토링
close #20
📌 Tasks
누락된 ProjectJoin 생성자 추가
Skill.name -> Skill.skillType (enum) 으로 수정
CareerInfo, CareerInfoRes -> MemberRes로 통합
Career를 계산하기 위해 ProjectMember에 의존해야 헀던 관계 제거
기존 MemberRes에서 CareerInfo, CareerInfoRes의 Career를 계산 방법을 그대로 적용
issue에서 언급했던 CareerInfo 여러 번 조회하던 코드 제거
총 경력 계산에서 사용하는 Period.plus는 총 경력이 12개월 이상이 되면 그대로 12M을 반환하던 문제 해결
총 경력을 모두 반환 -> 년차만 반환하도록 수정
ETC
어떤 의도로 작성한지 확실히 이해되지가 않아서 제 스타일대로 수정했습니다
혹시 놓친 부분이 있거나 문제가 생길 수 있는 부분은 피드백 주시면 감사하겠습니다
lsh2613
commented
1 week ago
member.loadCareer()를 service 계층에서 호출하여 dto에 전달하도록 수정
CareerInfoRes 적용
dto가 기능을 수행하지 않도록 수정
변경 사항 때문에 ProjectServiceTest.getSpecificProjectInfo_Success() 실패 떠서 처리해보려 하는데 하다가 막혀서 우선 주석처리 해뒀습니다..
일정이 있어서 좀 늦었네요
⭐ Summary
📌 Tasks
ETC