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

mutation 리팩토링 #146

Merged
merged 12 commits into from
Dec 13, 2020
Merged

mutation 리팩토링 #146

merged 12 commits into from
Dec 13, 2020

Conversation

GrasshopperBears
Copy link
Collaborator

해당 이슈 📎

#95

변경 사항 🛠

  • 택시 요청 시 requestingUsers에 존재하는 document 제거
  • 유저 회원가입 시 중복 확인
  • mutation을 user/driver/auth로 분리했습니다
  • schema에 input type을 추가했습니다
    • schema 변경에 따라 query를 변경했습니다

테스트 ✨

없음

리뷰어 참고 사항 🙋‍♀️

  • 팀원에게 하고 싶은 말

@GrasshopperBears
[be] refactor (#95) mutation 파일 분리
3abb777 
- mutation 폴더에 들어있던 resolver들을 역할에 맞게 분리
@GrasshopperBears
[be] refactor (#95) 유저가 요청 시 기존 requestingUser 제거
5251eff 
- requestingUser 정보가 누적되는 버그가 생겨 DB에서 제거하도록 변경
@GrasshopperBears
[be] refactor (#95) auth mutation 에러 핸들링 및 오류 메세지 분리
b74c651 
- try/catch로 에러  처리
- 에러 메세지 분리
@GrasshopperBears
[be] feat (#95) 중복 유저 확인
cfb2747 
- 승객의 id 중복 여부 확인
@GrasshopperBears
[be] refactor (#95) 공통 에러 메세지 분리
6129af5
@GrasshopperBears
[be] refactor (#95) 불필요한 graphql 스키마
52af16b
@GrasshopperBears
[fe] fix (#95) svg 속성 오타 수정
3e679b3
@GrasshopperBears
[common] refactor (#95) 드라이버 회원가입 입력 type 추가
0ac8cef 
- graphql 스키마에 추가
- 변경된 스키마에 맞춰 query 수정
@GrasshopperBears
[be] refactor (#95) 내역 저장 input 타입 추가
5eb8678 
- 스키마에 HistoryInfo 타입 추가
- 이에 따른 query 수정
@GrasshopperBears
[common] refactor (#95) signin input 타입 추가
06ca20b 
- 정보를 info 객체로 래핑
@GrasshopperBears
[common] refactor (#95) 유저 회원가입 input 타입 추가
68f9859 
- userSignup graphql 스키마 변경
- 이에 따른 query 변경
@GrasshopperBears
[be] refactor (#95) directive 위치 변경
3955dd6
@GrasshopperBears GrasshopperBears added this to the 4주차 milestone Dec 11, 2020
@GrasshopperBears GrasshopperBears self-assigned this Dec 11, 2020
ahnsoheee
ahnsoheee approved these changes Dec 11, 2020
View reviewed changes
Copy link
Contributor

@ahnsoheee ahnsoheee left a comment

Choose a reason for hiding this comment

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

mutation 나누고 싶었는데 진우님이 해주셨네요ㅎㅎ
수고 많으셨습니다

slogup-mia
slogup-mia approved these changes Dec 12, 2020
View reviewed changes
Copy link
Contributor

@slogup-mia slogup-mia left a comment •
edited
Loading

Choose a reason for hiding this comment

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

수고하셨습니다!
이번 리팩토링은 dev로 바로 넘기지 말고
refactor/merge 같은 브랜치 생성 브랜치들 합치면서 가면 어떨까요
파일이 삭제/생성된 부분이 많아서 꼬이면 브랜치 삭제해도 문제 없으니까요!

@GrasshopperBears GrasshopperBears changed the base branch from dev to refactor/merge December 13, 2020 07:07
@GrasshopperBears GrasshopperBears merged commit 17cd45b into refactor/merge Dec 13, 2020
@GrasshopperBears GrasshopperBears deleted the refactor/mutation branch December 14, 2020 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slogup-mia slogup-mia slogup-mia approved these changes

@ahnsoheee ahnsoheee ahnsoheee approved these changes

Assignees

@GrasshopperBears GrasshopperBears

Labels
🌈 client 🔧 refactor 🧱 server
Projects
None yet
Milestone
4주차
Development

Successfully merging this pull request may close these issues.
3 participants
@GrasshopperBears @slogup-mia @ahnsoheee