Skip to content

Conversation

@imtaejunk
Copy link
Collaborator

@imtaejunk imtaejunk commented Dec 6, 2024

image
지금 이미지 url 요청이 안 가고 있는데 원인을 모르겠습니다. 일단 올리고 나중에 마저 수정하겠습니다.

@github-actions
Copy link

github-actions bot commented Dec 6, 2024

💄 Storybook: https://673dcf13c9a3a1189a7c6fd4-fvxzhpqorb.chromatic.com/ # Chromatic에서 생성된 Storybook URL
🕖 Update: 2024년 12월 07일 06시 17분 50초 # 현재 시간

@hongggyelim
Copy link
Collaborator

workroot/src/app/(pages)/(addform)/addform/page.tsx
태준님 제가 작성한 코드 보시면 이미지 api 관련된 부분있어요!
이미지를 코드잇 api서버에 한번 업로드 한 다음 거기서 받아온 response를 formData에 넣어서 다시 보내줘야합니다.

alt="Kebab Menu Icon"
width={isLargeScreen ? 28 : 24}
height={isLargeScreen ? 28 : 24}
/>{" "}
Copy link
Collaborator

Choose a reason for hiding this comment

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

불필요한 공백 {" "} 이 들어간거같아요!


const Comment: React.FC<CommentProps> = ({ userName, date, comment }) => {
const Comment: React.FC<CommentProps> = ({ userName, date, comment, onKebabClick }) => {
const [isLargeScreen, setIsLargeScreen] = useState(false);
Copy link
Collaborator

Choose a reason for hiding this comment

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

useWidth 훅을 활용하셔도 됩니다

};
}, [setValue]);

const handleImageClick = () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

이미지 선택할때 또는 제출할때 image 업로드 api를 한번 거쳐서 url을 받아야합니다
PR comment 확인부탁드려요~

</form>
</div>
</div>
{/* 모바일 버전 버튼 */}
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

@hongggyelim hongggyelim left a comment

Choose a reason for hiding this comment

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

확인했습니다!

Copy link
Collaborator

@yyezzzy yyezzzy left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

@yyezzzy yyezzzy merged commit 7e65f6f into dev Dec 9, 2024
5 checks passed
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.

4 participants