Skip to content

Conversation

@gongsua
Copy link
Collaborator

@gongsua gongsua commented Oct 15, 2025

@gongsua gongsua requested a review from f-lab-ted October 15, 2025 10:13
@gongsua gongsua self-assigned this Oct 15, 2025
@gongsua gongsua added the documentation Improvements or additions to documentation label Oct 15, 2025
@gongsua gongsua linked an issue Oct 15, 2025 that may be closed by this pull request
@gongsua gongsua changed the title [#1] dosc/ [#1] dosc/API,ERD 업데이트 Oct 15, 2025
@sonarqubecloud
Copy link

@f-lab-ted
Copy link

[기타 의견]
README 파일 프로젝트 소개에 "이 프로젝트를 통해서 얻고자 하는 기술적 성취"가 정확히 무엇인지 알려주고 시작하면 좋을 것 같습니다.
완성된 하나의 서비스를 구현하는게 아니라 배움과 연습의 과정이기 때문에 목표를 명확히 드러내면 추후에 이 프로젝트로 윤진님의 실력을 평가하려는 분들에게 좋은 가이드가 될 것 같습니다.

@f-lab-ted
Copy link

f-lab-ted commented Oct 20, 2025

[ERD 리뷰]

  • 유저 테이블에 password 정보가 없어서 인증이 불가능할 것 같습니다.
  • 일부 테이블 간에 관계 표시를 2개씩 해둔 이유가 무엇인가요?
  • 게시물 테이블에서 "수정"에 대한 고려는 의도적으로 배제하신건가요?
  • 채팅방 테이블에서 회원id 필드는 단순히 게시물 작성자의 id를 입력하려는걸까요? 그렇다면, 데이터 중복 저장이 불필요하게 발생하는 것 같은데요. 채팅방 테이블과 채팅방 참여 회원 테이블을 별도로 두시는건 어떨까요? 채팅방은 1:1 채팅만 고려하고 계신지도 명확한 의도를 드러내는게 좋을 것 같습니다.
  • 게시물 상세내용 테이블과 게시물 테이블 존재의 의미는 하나의 게시물에 여러 상품을 올려둘 수 있으며, 각 상품마다 하나의 게시물 상세내용 레코드를 가지고 상세내용마다 자체 수량(필드)을 가진다는 걸로 이해하면 될까요? 그러면, 게시물 테이블의 가격(필드)랑 좀 안 맞는 것 같은데. 어떤 기획을 의도했고, 그 기획 의도에 맞춘 테이블 설계 과정과 결과를 함께 설명해주시면 좋을 것 같습니다.
  • 알람 테이블에서 채팅방id, 게시물id, 주문id 필드는 상황에 따라 null값이 들어갈 수도 있지 않을까요? notnull로 설정되어 있어서 물어봅니다. 만약 nullable 하다면, 차라리 JSONB 타입 데이터를 사용해보는 것도 고려해볼법 할 것 같아요.
  • 결제와 관련해서는 토스결제 테이블이라고 두면, 추후에 결제 방법 확장성 측면에 있어서 관리 이슈가 발생할 가능성이 커보입니다. 그리고 토스 결제 관련해서 참고한 문서들도 링크와 참고한 부분 모아서 정리해주시면 좋을 것 같아요.
  • 전반적으로 모든 테이블에서 soft delete / 수정에 대한 고려는 의도적으로 배제한 것일까요?

@f-lab-ted
Copy link

[API 명세 리뷰]

  • GET /api/main API는 정렬, 필터링 등에 대한 내용은 이 프로젝트에서 배제하려는 의도인건가요?
  • GET /api/users/chatrooms API의 응답 예시에 보면, lastMessage와 관련된 부분이 나오는데, 그럼 채팅방 목록은 RDB에서, 일부 채팅 내역은 MongoDB에서 불러와야하는데 이로 인한 읽기 작업의 비효율에 대해서는 고려해보셨을까요?
  • 인증과 관련해서는 지난번에 Session 방식을 고려하고 있다고 하셨는데, Token 방식으로 계획을 변경하신걸까요? Token 방식을 고려하고 있고 refresh token을 이용한 token rotate를 고려하고 있다면, 로그아웃 API에서는 refresh token을 이용하여 로그아웃 처리를 하는게 맞을 것 같습니다. 인증과 관련한 기술적 계획에 대해 조금 더 구체적으로 정리해주세요!
  • 채팅방 관련해서도 기획적인 부분을 좀 더 정리해주시면 API 명세를 이해하는데 도움이 될 것 같습니다. PRIVATE/OPEN 두 종류가 있는 것 같은데 각각에 대한 세부적인 기획 내용 업데이트 해주세요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Erd,Api 명세서 업뎃 밑 검토

3 participants