# sugn up, sign in transfer to jpa

[Maybaba]

적용하지 않는 기능임

개요

• sign-up, sign-in JPA 쿼리 및 엔터티 변환
• customer 도메인
• store 도메인
• auth 도메인
• 사용하지 않는 mapper 주입 코드 삭제

PR 유형

어떤 변경 사항이 있나요?

• 새로운 기능 추가
• 버그 수정
• CSS 등 사용자 UI 디자인 변경
• 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
• 코드 리팩토링
• 주석 추가 및 수정
• 문서 수정
• 테스트 추가, 테스트 리팩토링
• 빌드 부분 혹은 패키지 매니저 수정
• 파일 혹은 폴더명 수정
• 파일 혹은 폴더 삭제

작업사항

변경로직

mapper -> repository 쿼리로 의존

[hoseong511]

주석 처리하신 것들중에 앞으로 사용할 예정이 없는 것들이라면 전부 지워주세요.

[Maybaba]

@hoseong511 말씀해주신대로 주석 전부 지워달라 하셔서 지웠습니다 ~

[hoseong511]

jpa 전환 중에 사용하신 테스트 파일이 있으면 같이 올려주세요!

[Maybaba]

@hoseong511 호성님 전환완료한 jpa 파일에 테스트 파일은 들어있지 않아요. 프론트엔드측에서 로컬스토리지의 토큰을 같이 확인해야 해서 백엔드 자체에서 토큰을 발급하고 인증하는데 크게 의미를 두지 않았어요. 필요하거나 방법이 있다면 오늘 테스트를 만들어 올리겠지만
지금 저에게 드는 생각은 로그인을 해서 테스트를 돌리는 시간이 제가 코드를 만들어서 풀리퀘스트를 올리는 시간보다 빠르다고 생각이 드네요.

[Maybaba]

저희팀은 지금 남은 시간 안에 배포를 해야 하기에 안정성도 어느정도 중요하지만 발견하신 버그나 에러가 있다면 호성님의 경험도 많으시니 작업자와 함께 빠르게 현황을 확인하고 에러를 수정하는게 더 효율적이지 않을까요? 호성님의 역할이 기능병합, 테스트, 전체 총괄이라면 더욱더요.
물론 저도 잘 아는 것도 아니고 경험이 많은것도 아니지만 제가 느끼기에는 이 방법이 더 효율적이라 코멘트 남깁니다.

[hoseong511]

@hoseong511 호성님 전환완료한 jpa 파일에 테스트 파일은 들어있지 않아요. 프론트엔드측에서 로컬스토리지의 토큰을 같이 확인해야 해서 백엔드 자체에서 토큰을 발급하고 인증하는데 크게 의미를 두지 않았어요. 필요하거나 방법이 있다면 오늘 테스트를 만들어 올리겠지만 지금 저에게 드는 생각은 로그인을 해서 테스트를 돌리는 시간이 제가 코드를 만들어서 풀리퀘스트를 올리는 시간보다 빠르다고 생각이 드네요.

전환 중에 윤종님이 jpql로 작성한 쿼리를 검증한 테스트 파일이 있으시면 올려달라고 말씀드린거지 굳이 다시 테스트 파일을 만들어서 올려달라는 얘기는 아니였습니다.

저번에 메인 브랜치에서 빌드 문제가 발생해서 revert 처리한 거 기억하시는지요? 윤종님 뿐만 아니라 모든 분들에게 기능 개발한 사항에 대한 테스트 파일이 있는지 여쭤보고 있었고, 테스트 파일이 없으면 제가 확인을 하고 있었습니다.

말씀하신대로 로그인을 해서 전환해주신 jpql 쿼리를 확인해서 리뷰를 끝내보겠습니다. 감사합니다

[hoseong511]

저희팀은 지금 남은 시간 안에 배포를 해야 하기에 안정성도 어느정도 중요하지만 발견하신 버그나 에러가 있다면 호성님의 경험도 많으시니 작업자와 함께 빠르게 현황을 확인하고 에러를 수정하는게 더 효율적이지 않을까요? 호성님의 역할이 기능병합, 테스트, 전체 총괄이라면 더욱더요. 물론 저도 잘 아는 것도 아니고 경험이 많은것도 아니지만 제가 느끼기에는 이 방법이 더 효율적이라 코멘트 남깁니다.

언행중에 문제가 있는 것 같습니다. 애초 제가 경험이 많다고 이런 역할을 자처했다고 생각하시는것이 문제가 있는것 같습니다. 저도 개발을 하고 싶고 단지 어떻게 프로젝트를 잘 마무리할 수 있을지 고민을 하고 그런 고민들을 제안을 드릴뿐이지 경험이 많아서 에러를 수정하는 그런 역할을 하는 사람으로 보시는것에 기분이 나쁘네요