2020-09-09, 33th review meeting

[jongfeel]

249 comment 달아 주신 내용 정리

email 중복 처리의 경우

지금은 입력이 들어올 때 마다 계속해서 server에 요청하고 validation, overlapping 두 가지를 한꺼번에 처리합니다. 이걸 아래와 같이 나눠보면 어떨까 싶네요?

• email validation: client에서 입력할 때 마다 계속해서 처리, fetch 실행 안함, validation만 체크하므로 속도 빠름
• email overlapping: email validation에 통과 됐을 때 처리, fetch 실행, 이때 살짝 느린데 결국 email 입력을 완료한 상태에서 중복 체크만 하는 것이므로 어느 정도 느려도 상관 없음.

object or object field

• 이 경우는 취향대로 하시는게 맞는 것 같고
• 제 의견을 드리면 params.email 변수까지 가져와서 사용하는게 더 나아 보이긴 합니다.

email 정보 노출

• 우선 API 부터 변경해야 합니다. GET method를 쓴다는 건 server 쪽에서 뭔가 데이터를 조회해서 가져오는 용도로 쓰는데 지금은 client 쪽에 주는 데이터가 없이 url redirect만 하고 있습니다.
• 이 API를 POST method로 바꿔서 form data를 받고 처리하겠다는 의미로 변경하면, email 주소는 자연스럽게 form data로 넣어서 처리해 주면 해결되겠죠?

257 oauth login

• 기술적인 검토를 어느 정도 하신 것 같네요.
• 이걸 구현하기 위해서 예전에 @yoonucho 님에게 제가 훈련 시키기 위해 했던 분석, 설계 과정을 먼저 진행해 보면 좋을 것 같습니다.
• 지금은 기술적인 구현에 대한 분석을 하신 거고, 요구사항에 대한 분석이 추가로 들어가면 좋습니다.
• 설계의 경우도 기술적 분석 쪽에 예제를 통해 기능 구현에 대한 설계가 있으므로 이걸 냥터레스트 설계로 맞춰서 하면 됩니다. 데이터는 뭘 저장할 것이고 API는 뭐가 필요하고 어떻게 호출되서 흘러 갈 것인지를 정하는 것이죠.

[jongfeel]

Pull request를 잘 활용해 보기

• 문제로 적어주신 내용을 pull request로 잘 만들어서 그때 마다 issue에 링크로 등록하면 한번에 참조해서 볼 수 있는 문서를 만들 수 있고
• 역으로 이미 만들어진 issue에 pull request를 연결해서 html의 본래의 기능을 잘 활용할 수 있습니다.