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

[조현지] sprint10 #625

Conversation

cindycho0423
Copy link
Collaborator

@cindycho0423 cindycho0423 commented Jun 7, 2024

요구사항

기본

✨상품 등록 페이지

  • 상품 등록 페이지 주소는 “/addboard” 입니다.
  • 게시판 이미지는 최대 한개 업로드가 가능합니다.
  • 각 input의 placeholder 값을 정확히 입력해주세요.
  • 이미지를 제외하고 input 에 모든 값을 입력하면 ‘등록' 버튼이 활성화 됩니다.
  • 회원가입, 로그인 api를 사용하여 받은accessToken을 사용하여 게시물 등록을 합니다.
  • ‘등록’ 버튼을 누르면 게시물 상세 페이지로 이동합니다.

✨상품 상세 페이지

  • 상품 상세 페이지 주소는 “/addboard/{id}” 입니다.
  • 댓글 input 값을 입력하면 ‘등록' 버튼이 활성화 됩니다.
  • 활성화된 ‘등록' 버튼을 누르면 댓글이 등록됩니다

주요 변경사항

  • 랜딩페이지 / 로그인페이지 / 회원가입 페이지 가져왔습니다. css 까지 만질 시간은 없어서 그냥 회원가입과 로그인 기능 구현만 완료했습니다.

스크린샷

image image

멘토에게

  • 이미지를 binary 로 받아오는게 맞는지 궁금합니다. {,...} 형태로 받아오고, image: binary 라고 되어있는데 다른 사람은 formdata 형식으로 받아와지는 것을 보았습니다. 어떤 차이가 있는지 궁금합니다.
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@cindycho0423 cindycho0423 requested a review from ding-co June 7, 2024 08:10
@cindycho0423 cindycho0423 self-assigned this Jun 7, 2024
@cindycho0423 cindycho0423 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Jun 7, 2024
pages/boards.tsx Outdated Show resolved Hide resolved
lib/constants.ts Show resolved Hide resolved
lib/api/axios.ts Show resolved Hide resolved
lib/api/postComment.ts Show resolved Hide resolved
lib/utils/formatTime.ts Show resolved Hide resolved
components/submit-btn.tsx Show resolved Hide resolved
pages/login.tsx Show resolved Hide resolved
pages/signup.tsx Show resolved Hide resolved
components/articles.tsx Show resolved Hide resolved
}, [id]);

useEffect(() => {
const handleObserver = (entities: IntersectionObserverEntry[]) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

인피니티 스크롤 관련 부분에 대해서 로직만 별도 커스텀 훅으로 빼면 좋을 것 같아요.

Copy link
Collaborator

Choose a reason for hiding this comment

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

useIntersectionObserver 훅으로 한번 찾아보세요~

@ding-co
Copy link
Collaborator

ding-co commented Jun 9, 2024

이미지를 binary 로 받아오는게 맞는지 궁금합니다. {,...} 형태로 받아오고, image: binary 라고 되어있는데 다른 사람은 formdata 형식으로 받아와지는 것을 보았습니다. 어떤 차이가 있는지 궁금합니다.

보통 파일 업로드 시 이미지를 서버로 전송할 때 form data 형태를 많이 사용합니다.
받을 때는 이미지가 binary 로 내려오겠지만 만약 서버에서 form data 형식으로 내려준다면 클라이언트에서 받을 수는 있겠지만 일반적인 경우는 아닙니다. 질문 관련해서 스크린샷도 같이 첨부해주시면 파악하는데 더 좋을 것 같아요.

@ding-co ding-co merged commit 1a64151 into codeit-bootcamp-frontend:Next.js-조현지 Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants