Feat: 알바폼 상세 페이지 이미지 수정(수정 중) #137
Conversation
…/albaFormDetail
…lba/-formIdsrc/app/(pages)/(albaform)/alba/[formId]/edit/components/FormImage.tsx]-move
|
💄 Storybook: https://673dcf13c9a3a1189a7c6fd4-istdvbpevn.chromatic.com/ # Chromatic에서 생성된 Storybook URL |
cccwon2
requested review from
cccwon2,
hongggyelim and
imtaejunk
and removed request for
cccwon2
| if (!isOwnerRole) { | ||
| return ( | ||
| <div className="space-y-4 text-2xl"> | ||
| <Button className="h-10 sm:h-12 md:h-16" width="lg" icon={<FcFile />}> |
There was a problem hiding this comment.
"h-10 sm:h-12 md:h-16"와 같이 반복되는 코드는 별도 변수로 사용하는게 유지보수 측면에서도 좋다고 생각합니다!
There was a problem hiding this comment.
제가 폴더 합치면서 edit 밑에 components 폴더를 추가했네요! 상세 페이지와 관련된 컴포넌트는 alba/[formId]/component 경로로 두는게 더 좋을것같아요
There was a problem hiding this comment.
파일 이름에 오타가 났네욥 Info(r)mation
| <div className="flex h-[calc(100vh-200px)] items-center justify-center"> | ||
| <LoadingSpinner /> | ||
| </div> | ||
| ); |
| {albaFormDetailData && ( | ||
| <> | ||
| <FormHeader albaFormDetailData={albaFormDetailData} /> | ||
| <FormDetails albaFormDetailData={albaFormDetailData} /> |
There was a problem hiding this comment.
쿼리 데이터를 prop으로 넘겨줄 필요가 있을까요? 🤔 컴포넌트 내에서 데이터 값을 가져오는건 어떨까요? 잘 모르겠네용
There was a problem hiding this comment.
저도 쿼리를 호출하느냐 props로 넘겨주느냐 AI 선생님한테 물어봤는데 현재 코드는 깊이가 별로 안되어서 넘겨주는게 더 낫다고 하더라구요 👀
There was a problem hiding this comment.
아항 쿼리를 호출하는게 반드시 효율적인건 아닌가보네요! 👍
|
|
||
| import React from "react"; | ||
|
|
||
| export default function DotLodingSpinner() { |
There was a problem hiding this comment.
오타 잘 찾으시네요 ㅎㅋㅋ 👍
해야할 것
한 것