Skip to content
This repository has been archived by the owner on Sep 8, 2021. It is now read-only.

[yejeong, hyoon, sucho] - 피신 끝!!! #4

Open
wants to merge 29 commits into
base: master
Choose a base branch
from

Conversation

Yenowme
Copy link

@Yenowme Yenowme commented Aug 31, 2021

그래도 많이 배웠네요 👍

@cos18 cos18 requested a review from a team August 31, 2021 16:46
@cos18 cos18 requested review from nyeoni and jikwon101 and removed request for a team August 31, 2021 16:46
Copy link
Member

@nyeoni nyeoni left a comment

Choose a reason for hiding this comment

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

image
image
OAuth 로그인 동작원리에 대해서 정확하게 이해하고 계셨습니다. 제가 패스포트 동작원리가 헷갈리는 상태였는데 설명을 듣고 이해를 할 수 있었습니다. 그리고 인디언 포커 로직을 설명해주셔서 어떤 식으로 동작하려고 했고 클라이언트와 소켓통신을 어떻게 하려고 했는지 알 수 있었습니다. 친절하고 자세하게 설명해주셔서 감사해요!! 피신 정말 수고 많으셨습니다!!

Copy link

@jikwon101 jikwon101 left a comment

Choose a reason for hiding this comment

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

Screen Shot 2021-09-01 at 6 49 11 PM

Screen Shot 2021-09-01 at 6 49 55 PM

시간이 조금만 충분했더라면 완전한 결과가 나왔을 것 같은데 너무 아쉽습니다. 수고하셨습니다 !

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

Successfully merging this pull request may close these issues.

5 participants