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

[민지영] sprint8 #120

Conversation

Obsessive-Curiosity
Copy link
Collaborator

@Obsessive-Curiosity Obsessive-Curiosity commented Mar 2, 2025

배포링크: https://panda-market-drfxcgudc-min-jiyeongs-projects.vercel.app/article

요구사항

기본

  • [x]
  • []
  • []

심화

  • [x]
  • []

주요 변경사항

  • 리액트 쿼리의 invalidatequeries를 해도 바로바로 업데이트가 안되네요 ㅠ 로컬에서는 됐었는데 나중에 next 캐시로 수정해보겠습니다.

스크린샷

image

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

Header와 Footer 설정
이미지 추가
자유게시판 페이지 완성
서버액션 구현 완료
불투명한게 간지나니깐!
수정 및 삭제 기능 미완성
파일로 분할
변수명 변경
게시글 작성시 리액트 쿼리 캐싱 무효화 시키기 추가. SWR형식이라서 게시글 작성후 게시판으로 돌아가면 리액트 쿼리가 새로고침 되지 않음 문제 발생하기 때문에 리팩토링 함
Brand타입 사용 및 코드 개선
게시글 수정 기능 추가
게시글 기능 추가!
Copy link
Collaborator

@dennis2311 dennis2311 left a comment

Choose a reason for hiding this comment

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

잘 하셨습니다

  • 주석! 주석을 많이 사용해주세요. 이 정도 규모에서 주석이 없다면 아마 몇 주 뒤에 자신의 코드를 디버깅할 때 많은 어려움을 겪을 수 있습니다. (특히 같이 일하는 사람들이..)
    • 이 때 JSDoc을 활용해보시는 것을 강력 권장해 드립니다.
  • 코드의 규칙성을 찾아 공통된 지점을 묶고 모듈화하는 것에 대단히 능숙하시네요. fetch 요청을 종류별로 모듈화 한 부분은 저도 많이 감명을 받았습니다. 솔직히.. 정말 많이 놀랍습니다.
    • 다만 이렇게 섬세하게 짜여진 타이핑과 유틸 함수들에 주석이 없다면 협업할 때 오히려 장애물로 작용할 수 있습니다. 모두가 그 규칙을 숙지하고 잘 활용할 수 있다면 안정적이고 효율적인 코드가 되지만, 그렇지 않다면 나 혼자 잘난 코드가 되어버릴 수도 있습니다.
  • 작성한 코드를 어떻게 문서화하고 남에게 잘 설명할 수 있을지 고민하는 단계이신 것 같네요

배포 설정
context export import 수정
update
@Obsessive-Curiosity
Copy link
Collaborator Author

답변 감사합니다!
다음부턴 JSDoc 활용하여 주석도 열심히 달도록 하겠습니다!
fetch 요청은 next 캐싱 뿐만 아니라 react query에서도 적용되게 하느라 많이 수정되어 나중에 또 변경될 수도 있습니다.
그리고 배포링크도 올려두었습니다!

@dennis2311 dennis2311 merged commit f4c6d77 into codeit-sprint-fullstack:next-민지영 Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants