-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Fix] #33 포스트 이미지 보이게 수정 #37
base: main
Are you sure you want to change the base?
The head ref may contain hidden characters: "fix#33/\uD3EC\uC2A4\uD2B8-\uC774\uBBF8\uC9C0-\uBCF4\uC774\uAC8C-\uC218\uC815"
Conversation
}; | ||
}; | ||
|
||
export default function PostImages({ post }: Props) { | ||
if (!post?.Images || post?.Images.length === 0) return null; | ||
if (!post?.images || post?.images.length === 0) return null; |
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.
이미지가 없는 경우에 대한 ui 반응은 따로 없는 건가요?
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.
이미지가 없는 경우는 PostImage 컴포넌트가 안떠서 Post 컴포넌트에 그냥 글만 뜨게 되어 있어요!
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.
확인 했습니다!
}; | ||
}; | ||
|
||
export default function PostImages({ post }: Props) { | ||
if (!post?.Images || post?.Images.length === 0) return null; | ||
if (!post?.images || post?.images.length === 0) return null; |
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.
확인 했습니다!
1. 무슨 이유로 코드를 변경했나요?
포스트에서 api response를 처리할 때 데이터 구조가 잘못 되어 있어서 이미지 url을 받아오지 못했습니다.
이를 수정하여 이미지를 불러올 수 있도록 고쳤습니다.