Skip to content

Feature/#7 signup with google#23

Merged
2rebi merged 10 commits into
developfrom
feature/#7-signup-with-google
Jul 28, 2021
Merged

Feature/#7 signup with google#23
2rebi merged 10 commits into
developfrom
feature/#7-signup-with-google

Conversation

@2rebi
Copy link
Copy Markdown
Member

@2rebi 2rebi commented Jul 28, 2021 •
edited
Loading

Description
필수 리뷰어: @RbertKo

진행 배경

작업 목표

  • 구글 인증으로 회원가입을 한다.
  • 구글 인증으로 로그인을 한다.

결과

변경 후

  • 회원가입 API / 스웨거 문서를 기반으로 API 테스트 쌉가능
  • Feign Client 의존성 추가
  • Feign Client 관련 설정 파일 및 Config DI 추가
  • User Service -> 유저 생성 구현(구글만)
  • 컨트롤러에서 API 호출시 웹플럭스 모노로 받고 validation 핸들링 부분 추가

어려웠던 점

  • User Service에서 코루틴 사용시 초반에 CoroutineScope 를 사용하여 작업하니 예외 발생시 롤백이 안되는 점
    • runBlocking 으로 해결(코틀린 코루틴 관련 깊은 학습이 필요)

레퍼런스

기타

  • TODO로 적어놓은 Exception 부분들 모두 정의 및 Exception Controller 에서 핸들링하여 알맞는 에러 형태로 변환

2rebi added 10 commits July 29, 2021 00:20
@2rebi
chore: feign dependency added and config yaml file modified for feign…
88a8153 
… client
@2rebi
chore: 쓸때 없는 파일 제거 및 불 필요한 브라켓 제거
bc30cb9
@2rebi
feat(feign-setup): feign client enabled, feign config defined
55b65ac
@2rebi
feat(domain/social-provider): social provider repository, adapter, so…
b0e92b9 
…cial info interface defined
@2rebi
feat(infra/feign-social-provider): implements social adapter(google) …
ec4f338 
…/ info data class / define feign client interface, google get token info api
@2rebi
feat(app/exceptions): validation exception webflux mono handler exten…
a4d2cf8 
…sion function added
@2rebi
chore: 불필요한 임포트 제거
e7bce73
@2rebi
feat(google-signup): bind controller
0099349
@2rebi
feat(user-service): create user with google
09db9dc
@2rebi
fix(social-provider): idclass -> embeddedId, because if use idClass w…
91c662c 
…ill add to some two more columns.

문법 몰라으ㅏㅇ그에그덱ㄷ게ㅡㄴㅇㅁ라 영어 잘하고싶습니다 영어 과외 해주실분 구해요
영어 스터디 구해요

연락주십셔
@2rebi 2rebi requested a review from RbertKo July 28, 2021 16:01
@2rebi 2rebi linked an issue Jul 28, 2021 that may be closed by this pull request
[STORY] User는 Google 계정을 이용하여 회원가입 및 로그인 할 수 있습니다. #7
Closed
1 task
RbertKo
RbertKo approved these changes Jul 28, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@RbertKo RbertKo left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏼

@2rebi 2rebi merged commit 43b8b48 into develop Jul 28, 2021
@2rebi 2rebi deleted the feature/#7-signup-with-google branch July 28, 2021 16:38
@2rebi 2rebi restored the feature/#7-signup-with-google branch July 28, 2021 16:38
@2rebi 2rebi deleted the feature/#7-signup-with-google branch August 3, 2021 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RbertKo RbertKo RbertKo approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[STORY] User는 Google 계정을 이용하여 회원가입 및 로그인 할 수 있습니다.

2 participants

@2rebi @RbertKo