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

[장수경] sprint11 #333

Conversation

Sookyeong02
Copy link
Collaborator

@Sookyeong02 Sookyeong02 commented Dec 8, 2024

요구사항

기본

  • [x]
  • []
  • []

심화

  • [x]
  • []

주요 변경사항

스크린샷

image

멘토에게

  • MarketPage 상세페이지([id].tsx)에서 404에러가 발생합니다.
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

Copy link
Collaborator

Choose a reason for hiding this comment

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

getServersideProps 를 완전히 제거하셨군요~?
이제 board 페이지에서는 에러 안 발생하나요??

Copy link
Collaborator

@baeggmin baeggmin left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~👏

MarketPage 에서 발생한다는 404 에러는 accesstoken 때문 아닐까요?
먼저 로그인 기능을 구현하시고 accesstoken 을 받아 request header 에 넣어주시면 해결될 듯 합니당

try {
// 올바르게 URLSearchParams 생성
// const query = new URLSearchParams().toString(); // 빈 쿼리 문자열을 생성
const query = `limit=${limit}`;
// const query = `limit=${limit}`;
const query = new URLSearchParams(params).toString();
Copy link
Collaborator

Choose a reason for hiding this comment

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

URLSearchParams 를 잘 활용해주셨습니다! 👍

@@ -1,6 +1,15 @@
/** @type {import('next').NextConfig} */
const nextConfig = {
reactStrictMode: true,
images: {
domains: [
Copy link
Collaborator

Choose a reason for hiding this comment

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

외부에서 이미지를 가져올 일이 있었나요..?

const [content, setContent] = useState("");

// 이미지 제외 모든 input에 값이 입력
const isSubmit = !title.trim() || !content.trim();
Copy link
Collaborator

Choose a reason for hiding this comment

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

사소한 코멘트지만 isDisabled 가 더 적절할 것 같습니당

@baeggmin baeggmin merged commit 802d3b7 into codeit-bootcamp-frontend:Next-장수경 Dec 10, 2024
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