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.

Sign up for GitHub

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

Jump to bottom

fix: 회원 탈퇴 시 탈퇴자 댓글 예외 처리 #316

Merged
merged 8 commits into from
Oct 22, 2024
Merged

fix: 회원 탈퇴 시 탈퇴자 댓글 예외 처리 #316

merged 8 commits into from
Oct 22, 2024

Conversation

char-yb
Copy link
Member

@char-yb char-yb commented Oct 19, 2024

🌱 관련 이슈

📌 작업 내용

  • comment-record mapping에 대한 연관관계 삭제
  • recordId를 저장하며 직접 proxy접근이 아니도록 수정
  • 회원탈퇴 시 comment 테이블의 writerId는 null로 처리되도록 수정

🙏 리뷰 요구사항

📚 레퍼런스

@char-yb char-yb added 🐛 bug/error 버그 및 에러 픽스 ♻️ refactor 기능 개선 🥇 P1 급하고 꼭 필요한 이슈 labels Oct 19, 2024
@char-yb char-yb added this to the 4차 MVP milestone Oct 19, 2024
@char-yb char-yb self-assigned this Oct 19, 2024
@char-yb char-yb linked an issue Oct 19, 2024 that may be closed by this pull request
🐛 회원탈퇴 시 댓글 데이터 삭제 #315
Closed
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Oct 21, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@char-yb char-yb merged commit ec66a63 into develop Oct 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dbscks97 dbscks97 Awaiting requested review from dbscks97 dbscks97 is a code owner

Assignees

@char-yb char-yb

Labels
🐛 bug/error 버그 및 에러 픽스 🥇 P1 급하고 꼭 필요한 이슈 ♻️ refactor 기능 개선
Projects
None yet
Milestone
4차 MVP
Development

Successfully merging this pull request may close these issues.

🐛 회원탈퇴 시 댓글 데이터 삭제
1 participant
@char-yb