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.

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

feat(member): move github username from assignment to member #749

Merged
merged 14 commits into from
May 21, 2024

Conversation

woowahan-neo
Copy link

@woowahan-neo woowahan-neo commented May 16, 2024

Resolves #744, resolves #752

해결하려는 문제가 무엇인가요?

  • 현재는 과제 제출물을 제출할 때 GitHub 사용자 이름을 받게 되는데, 회원 가입 시 받도록 변경한다.
  • 회원 가입 시 입력하는 필드가 계속해서 추가되고 있으니 이번 기회에 정리한다.
  • 전화번호, 휴대폰 번호, 휴대전화 번호휴대전화 번호로 표현을 통일한다.

어떻게 해결했나요?

  • 회원 가입 시 GitHub 사용자 이름을 받는 부분을 추가하고, 과제 제출물 제출 시 GitHub 사용자 이름을 받는 부분을 삭제하였습니다.
  • 회원 가입 시 입력하는 정보의 순서를 재정렬하였고, 관련 코드의 매개변수 순서도 통일하였습니다.

어떤 부분에 집중하여 리뷰해야 할까요?

  • 와이어프레임에 맞게 구현이 되었는지 확인해 주세요.
  • 의도한대로 작동하는지 확인해 주세요.

참고 자료


RCA 룰

r: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
c: 웬만하면 반영해 주세요. (Comment)
a: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

Copy link

@woowahan-cron woowahan-cron left a comment

Choose a reason for hiding this comment

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

"휴대전화 번호" 키워드가 프론트엔드 영역에서 변경된 것을 확인하였습니다 🙂
회원가입/로그인까지 시도해봤는데 동작에 이상 없는 것을 확인했습니다!
미리 챙겨주셔서 감사합니다!

@woowahan-pjs woowahan-pjs changed the title feat(member): add github username feat(member): move github username from assignment to member May 21, 2024
@woowahan-pjs woowahan-pjs merged commit c1dbac1 into develop May 21, 2024
@woowahan-pjs woowahan-pjs deleted the feature/github-username branch May 21, 2024 05:10
Copy link

@woowabrie woowabrie left a comment

Choose a reason for hiding this comment

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

코드 보는 사이 머지되었군요,,
로컬에서 확인 시 이상 없었습니다!

@@ -242,6 +242,7 @@ class DatabaseInitializer(
email = "[email protected]",
phoneNumber = "010-0000-0000",
gender = Gender.MALE,
githubUsername = "jaeyeonling",

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants