Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

218 댓글 신고 crud #219

Open
wants to merge 30 commits into
base: develop
Choose a base branch
from
Open

218 댓글 신고 crud #219

wants to merge 30 commits into from

Conversation

eugene225
Copy link
Collaborator

🧙 PR 타입

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

🤨 Motivation

Resolved #218

🔑 Key Changes

  • 댓글 신고 CRUD 구현
  • 댓글 신고 CRUD 테스트 코드 작성

🙏 To Reviewers

테스트 코드를 작성하다가 reply 엔티티에 테이블명이 mogakko로 되어 있던걸 발견했습니다..! 지금 데이터베이스 테이블 연관관계가 모두 꼬여있습니다ㅜ 이번 pr 머지하기전에 데이터베이스 한번 갈아야 할 것 같습니다
Screenshot 2024-07-18 at 12 03 36 AM
reply 테이블이 생성되지 않은 상태

@eugene225 eugene225 added feat 기능 추가 시 사용 test 테스트 코드 작성 labels Jul 17, 2024
@eugene225 eugene225 requested a review from ZZAMBAs July 17, 2024 15:09
@eugene225 eugene225 self-assigned this Jul 17, 2024
@eugene225 eugene225 linked an issue Jul 17, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 추가 시 사용 test 테스트 코드 작성
Projects
None yet
Development

Successfully merging this pull request may close these issues.

댓글 신고 CRUD
1 participant