-
Notifications
You must be signed in to change notification settings - Fork 17
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 #125
Merged
dennis2311
merged 16 commits into
codeit-sprint-fullstack:next-박세정
from
celina823:next-박세정-sprint8
Mar 5, 2025
The head ref may contain hidden characters: "next-\uBC15\uC138\uC815-sprint8"
Merged
[박세정] sprint8 #125
dennis2311
merged 16 commits into
codeit-sprint-fullstack:next-박세정
from
celina823:next-박세정-sprint8
Mar 5, 2025
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dennis2311
reviewed
Mar 4, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
잘 하셨습니다.
- 주석! 주석을 적극적으로 활용해 주세요! JSDoc 형식을 활용할 것을 강력히 권장합니다.
- 타입 정의, 데이터 fetching, 데이터 정합성 관리 등, 각 기능별로 그룹을 나누고 모듈화하는 힘을 길러주세요.
- freeboard.tsx 파일에서 해당 필요성이 잘 드러나고 있는데, CustomDropdown을 import 하여 사용한 것을 제외하면 데이터 조회, 타입 정의, 날짜 표시 방식 변경을 위한 util 함수 등 다른 곳에서 재사용 될 수 있거나, 재사용 되지 않더라도 개념적으로 분리되는 것이 적절한 내용들이 많이 포함되어 있어 파일의 크기가 커지고 가독성이 떨어집니다.
- 가능한 하나의 파일은 하나의 기능을 담당하도록 하여, 핵심적인 부분만 스스로 처리하고 다른 기능들은 다른 파일들에 '외주'를 맡길 수 있도록 모듈화를 연습해보시면 좋을 것 같습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
※ 무한스크롤 구현안됨 + 디자인 수정중으로, 완료 시 이 문장 삭제하겠습니다 !→디자인 수정은 완료하였으나 무한스크롤 구현이 안되어서 전체 게시글 데이터가 한번에 다 보이게 해두었습니다.ㅠ
※ render배포 시 계속 오류가 나서 해결되는대로 배포링크 공유드리겠습니다.→ npm run dev시에는 로컬에서 페이지가 잘 보이는데, build에서 수많은 오류가 나서 배포링크를 못올렸습니다.배포 성공했습니다!
배포링크: https://five-sprint-mission-fe.onrender.com
요구사항
기본 요구사항
공통
자유 게시판 페이지
→ 백엔드에서 임의 숫자 집어넣음. 새로 게시글 생성 시 기본값 0
게시글 등록 & 수정 페이지
게시글 상세 페이지
심화 요구사항
공통
마이그레이션에 상당한 시간이 소요될 수 있으므로 진행을 권장하지 않습니다.
주요 변경사항
스크린샷
멘토에게