기능구현 - 유저정보CRUD(Read)

[JunYoungee]

개요

유저정보 CRUD기능 중 Read 구현

구현

• Read 페이지에서는 User의 회원정보를 불러옴
• Read 버튼을 눌렀을 때 사용자의 회원정보를 불러옴
  -> 정상적으로 불러왔을 때 정상적으로 처리됐다는 문자를 console에 출력 후 회원정보를 페이지에 나열함
  -> 정상적으로 불러와지지 않았을 때 에러를 console에 출력함

[98StarJune]

개요

현재 본 브랜치의 이름은 read 로 설정되어 있습니다. 현재의 브랜치 명은 정확하게 어떤 작업을 진행 중인지 알기 쉽지 않습니다.

제안사항

현재 멤버님께서 진행해주신 작업은 User 관련 Read 작업이므로 feat/user_read 와 같이 브랜치 명을 의미있게 만들 수 있습니다.

[98StarJune]

제안합니다

현재 멤버님께서 여러 수정사항이 있었음에도 불구하고 커밋이 1개입니다. 문제가 되지는 않으나 추후 개발 중 문제가 발생했을때 피해를 최소화 하기 위해서 커밋의 단위를 더욱 작게 설정할 것을 제안드립니다.

[98StarJune]

코드 리뷰입니다

귀하의 노고에 진심으로 감사드립니다. 본 PR에 대한 리뷰를 다음과 같이 남겨드립니다.

제안 사항에 대한 수정

제안 사항으로 표시된 리뷰는 수정 제안을 드리는 것으로 타 파트원 및 다른 운영진의 의견을 첨언 받아 최종 결정하시기 바랍니다.

[98StarJune]

인터페이스 명칭 구체화 제안

현재 인터페이스의 이름이 Database로 설정되어 있으나 Database라는 단어를 통해 본 인터페이스가 무엇을 나타내는지 확인하기 어렵습니다. 의미있는 명칭으로 사용할 것을 추천드립니다.

[98StarJune]

merge 시 충돌이 예상됩니다

현재 수정된 package.json과 page.tsx는 해당 파트의 모든 멤버가 동시에 수정한 것으로 판단됩니다. 이 경우 merge 시에 충돌이 발생할 수 있습니다.

제안

이런것과 유사하게 모두에게 적용되어야하는 것이 있을 경우 별도의 이슈로 분리하고 develp에서 별도로 분기한 브랜치에서 작업하시는 것을 추천드립니다.