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

Fix comments count #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix comments count #120

wants to merge 1 commit into from

Conversation

MoonHyuk
Copy link

#95 에 관한 커밋입니다.
유저 프로필에서 다음과 같은 문제가 있었습니다.

  1. 쓴 글 수가 올바르게 나오지 않음
  2. 코멘트 수가 삭제한 코멘트까지 포함됨

댓글을 삭제 할 때 posts는 차감되는데, comment_count는 바뀌지 않아 발생하는 문제인것 같습니다.
is_removed=False로 삭제되지 않은 코멘트들의 수만 셈으로써 해결할 수 있습니다.
(개발환경을 세팅하지 못해서 테스트는 못해봤습니다. 😢 꼭 검증해보세요)

그리고, 한 게시물에서 같은 유저가 같은 내용의 코멘트를 달 때 코멘트가 정상적으로 달리지 않고 posts가 차감되는 현상이 있습니다.

https://stackoverflow.com/questions/34237451/same-content-for-same-user-and-same-post-in-django-comments-not-getting-saved

위 답변을 참고하시면 좋을것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant