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

11주차 과제 제출 #25

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

11주차 과제 제출 #25

wants to merge 1 commit into from

Conversation

ahyeon-github
Copy link
Member

📌 관련 이슈

#24

✨ 과제 내용

  • 11주차 과제 검색기능
  • 댓글기능 수정 / 삭제 기능 구현하기

📸 스크린샷(선택)

스크린샷 2022-07-18 오후 10 04 12

스크린샷 2022-07-18 오후 11 50 07

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

수정하기 버튼을 누르면 comment_update.html 로 넘어가지는데 페이지에서 수정한 내용이 반영이 안되어서 어떻게 코드를 수정해야 할지 모르겠습니다..!

@ahyeon-github ahyeon-github requested a review from a team July 18, 2022 15:50
@ahyeon-github ahyeon-github self-assigned this Jul 18, 2022
Copy link
Member

@soeunkk soeunkk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

검색기능 잘 구현하신거 확인했습니다!! 댓글 구현 과제에는 피드백 남겼으니 확인해주세요!!
과제가 좀 많았는데 수정,삭제까지 시도해보느라고 완전 수고많았습니다💙

return render(request,'diaryapp/comment_update.html', {'form':form})

def delete_comment(request, diary_id) :
post = get_object_or_404(Diary, pk=diary_id)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

객체를 받아올 때 Comment가 아닌 Diary 모델에서 검색하기 때문에 삭제 대상이 블로그 글이 돼요!
(그래서 현재 웹페이지에서 댓글을 삭제하면 댓글이 작성된 글이 삭제되었던 거예요.)
댓글을 삭제하려면 Diary 대신 Comment 모델로 바꿔주면 됩니다~!
그리고 변수명은 댓글과 관련된 것이니 diary_id 대신 comment_id가 좋겠죠?!
(urls.py에서도 변수명을 바꿔줘야 에러가 나지 않아요!)



def update_comment(request, diary_id):
post = get_object_or_404(Diary, pk=diary_id)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

여기도 마찬가지로 Diary 대신 Comment로 바꾸면 댓글 수정하는 페이지가 잘 뜰거예요!

Comment on lines +95 to +96
comment = form.save(commit=False)
comment.diary = diary
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instance를 통해 CommentForm을 가져왔기 때문에 이미 diary 정보도 포함되어 있어요!
그래서 95,96번째 줄은 삭제해도 좋습니다!

comment = form.save(commit=False)
comment.diary = diary
form.save()
return redirect('detail', diary_id=post.pk)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

91번째 line에서 모델을 Comment로 바꾸면 comment 변수에서 post의 pk값을 얻어와야해요!
diary_id=comment.diary.pk로 사용하면 됩니다~!!!!

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.

2 participants