Skip to content
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.

Sign up for GitHub

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

Jump to bottom

refactor - AuthFacade 관련 코드 리팩토링 #178

Merged
merged 19 commits into from
Jul 17, 2024
Merged

refactor - AuthFacade 관련 코드 리팩토링 #178

merged 19 commits into from
Jul 17, 2024

Conversation

kseysh
Copy link
Member

@kseysh kseysh commented Jul 16, 2024

Related issue 🚀

Work Description 💚

  • AuthFacade 관련 코드가 가독성이 너무 떨어지는 것 같아 코드 리팩토링을 진행하였습니다.

PR 참고 사항

  • challengeApp이 존재하는지 확인하기 위해 사용하는 challengeAppRepository의 existBychallengeeIdAndAppCodeAndOS를 한방쿼리로 Querydsl로 해보려고 gradle에 추가를 해놨는데 안되는 방식인 것 같아서.. 일단은 challengeAppRepository 추상화와 Querydsl 추가만 해두었습니다! 나중에 Querydsl로 sql 쿼리 개선도 진행해볼게요.

kseysh added 19 commits July 15, 2024 23:04
@kseysh
refactor - #174 불필요한 JsonProperty 삭제
b66c89d
@kseysh
refactor - #174 user validate를 addUser 메서드가 하도록 변경
08989a6
@kseysh
refactor - #174 챌린지 생성과 User에 currentChallengeId를 업데이트하는 로직이 같이 작동하도록 수정
fe333c8
@kseysh
refactor - #174 addChallenge메서드 네이밍을 하는 일과 적합하도록 변경
ef34a85
@kseysh
refactor - #174 챌린지 네이밍 수정
4577419
@kseysh
refactor - #174 로그인 메서드 파라미터 수정 및 가독성 개선을 위한 줄바꿈
57d37bf
@kseysh
modify - #174 이전 챌린지 앱을 가져오는 로직 단순화
02f72d9
@kseysh
modify - #174 챌린지를 처음 시작하는 메서드와 이전 챌린지를 이용하여 챌린지를 생성하는 로직으로 분리
fee93d8
@kseysh
refactor - #174 challenge에서 startDate를 가져오는 로직 dailyChallengeService로 이동
31c0fc6
@kseysh
feat - #174 유저를 이용해 currentChallengeId를 가져오는 메서드 추가
0d24b30
@kseysh
feat - #174 ChallengeRequest를 생성하는 로직 위치 변경
972a762
@kseysh
feat - #174 앱 저장 로직 변경
7b6ecc9
@kseysh
modify - #174 삭제 예정 유저 recover 로직 다른 메서드로 책임 이동
ee7d5de
@kseysh
refactor - #174 AuthFacade 로직 개선
a12962f
@kseysh
refactor - #174 사용되지 않는 메서드 제거
6513a46
@kseysh
refactor - #174 사용되지 않는 implementation 제거
761e34b
@kseysh
refactor - #174 challengeAppRepository 추상화
f62efcb
@kseysh
refactor - #174 비즈니스 로직인 에러처리를 service에서 책임을 갖도록 수정
3251a18
@kseysh
feat - #174 Querydsl에 필요한 build.gradle 설정 추가
cbbffcb
@kseysh kseysh added the 🔨 Refactor 코드 리펙토링 (기능 변경 없이 코드만 수정할 때) label Jul 16, 2024
@kseysh kseysh requested a review from jumining July 16, 2024 06:15
@kseysh kseysh self-assigned this Jul 16, 2024
jumining
jumining approved these changes Jul 16, 2024
View reviewed changes
Copy link
Collaborator

@jumining jumining left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리팩토링 수고하셨습니다! 작업하면서 다시 챌린지 관련 파일 또 살펴볼게요

@kseysh kseysh merged commit 1fd02be into develop Jul 17, 2024
1 check passed
@jumining jumining deleted the fix/#174-signup-npe branch November 19, 2024 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jumining jumining jumining approved these changes

Assignees

@kseysh kseysh

Labels
🔨 Refactor 코드 리펙토링 (기능 변경 없이 코드만 수정할 때)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix - 회원 가입시 null값인 currentChallengeId를 가져오는 문제
2 participants
@kseysh @jumining