Skip to content

Sangun week10 hw#53

Open
gse1818 wants to merge 3 commits intosangun-week10from
sangun-week10-hw
Open

Sangun week10 hw#53
gse1818 wants to merge 3 commits intosangun-week10from
sangun-week10-hw

Conversation

@gse1818
Copy link

@gse1818 gse1818 commented May 27, 2024

💎 과제 구현 설명

과제를 위해 사용했던 함수, 수정한 파일 등 구현 내용에 대한 간단한 설명을 작성해 주세요.

  • 기본적인 댓글 작성, 수정, 삭제 구현
  • 로그인 하지 않은 상태에서 PostDetailPage를 들어가면 댓글 작성란이 보이지 않도록 구현
  • 본인이 작성한 댓글이 아니면 수정, 삭제 버튼이 보이지 않도록 구현
  • if-else문에 들어가기 전에 interceptor에서 에러를 발생시키는 문제가 있어 axios.js를 수정
  • 세미나 하면서 작성한 Homepage.jsx에 중복코드가 있어서 제거

🏁 PR 체크리스트

  • [V] 코드가 오류 없이 정상적으로 실행되나요?
  • [V] 커밋 메시지 컨벤션(템플릿 활용)을 준수했나요?
  • [V] 과제 마감기한을 준수했나요?

🖼️ Screenshot / Video

-댓글 기능 구현
스크린샷 2024-05-27 오후 7 15 26
스크린샷 2024-05-27 오후 7 15 37
스크린샷 2024-05-27 오후 7 14 59

-로그인 하지 않은 상태에서는 댓글 작성란이 보이지 않음
스크린샷 2024-05-27 오후 7 14 44

-본인이 작성하지 않은 댓글은 수정, 삭제 버튼이 보이지 않음
스크린샷 2024-05-27 오후 7 15 10

🙌 Issue

과제 수행 중 어려웠던 부분이나 궁금했던 점을 자유롭게 작성해 주세요.

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