Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
sohyun0
left a comment
sohyun0
left a comment
There was a problem hiding this comment.
작업하느라 고생많으셨습니다! 코멘트는 제안 및 궁금증 정도여서 바로 승인하겠습니다 😃
| const { mutate } = useMutation({ | ||
| mutationFn: deleteTeam, | ||
| onSuccess: () => { | ||
| router.replace("/"); | ||
| }, | ||
| }); |
| return ( | ||
| <Container className="mt-[72px] max-w-[460px] pt-0 tablet:mt-[100px] tablet:px-0 desktop:mt-[140px] desktop:pt-0"> | ||
| <h2 className="mb-[24px] w-full text-center text-[24px] tablet:mb-[80px]">팀 수정하기</h2> | ||
| <form method="PATCH" className="w-full" onSubmit={handleSubmit}> |
There was a problem hiding this comment.
form method는 get/post만 있는걸루 알고있는데 혹시 추가하신 다른 이유가 있을까용!?:D
There was a problem hiding this comment.
머라고요 제가 바보같은 행동을 했군요... 기본적인것까지 까먹었네요ㅠ_ㅠ 알려주셔서 감사합니다
|
|
||
| updateGroupMutate.mutate({ groupId: groupId, payload: newData }); | ||
| }, | ||
| onError: error => console.log(error.message), |
There was a problem hiding this comment.
추후 작업하실때 60번 라인과 함께 토스트 띄워줘도 좋을것 같습니다 !
토스트 컴포넌트가 안보여서 전체적으로 나중에 작업하실거라고 예상이되긴하네용!
There was a problem hiding this comment.
넵! 미리 작업한거 먼저 올리고 한번에 토스트 작업하면 깔끔할것 같아서 그렇게 하려구 합니당
| {formData.image ? ( | ||
| <Image | ||
| src={formData.image} | ||
| alt="팀 프로필 사진" | ||
| className="inline-block h-[64px] w-full rounded-full border-[2px] border-line-strong" | ||
| width={64} | ||
| height={64} | ||
| /> | ||
| ) : ( | ||
| <IcProfile className="h-[64px] w-full" /> | ||
| )} | ||
|
|
||
| <IcEdit className="absolute bottom-0 right-0 h-[20px] w-[20px]" /> |
There was a problem hiding this comment.
여유가 되신다면 수정 -> 이미지 업로드 후 삭제하는 부분도 있으면 좋을것 같아요! :D
There was a problem hiding this comment.
네! 소현님이 x 해주신거 보고 이런방법이 있구나 알게되었어요 qa 후다닥 해서 구현해볼게요!
| return ( | ||
| <Container className="mt-[72px] max-w-[460px] pt-0 tablet:mt-[100px] tablet:px-0 desktop:mt-[140px] desktop:pt-0"> | ||
| <h2 className="mb-[24px] w-full text-center text-[24px] tablet:mb-[80px]">팀 수정하기</h2> | ||
| <form method="PATCH" className="w-full" onSubmit={handleSubmit}> |
There was a problem hiding this comment.
저희 공통 폼/검증 컴포넌트(<Form> + zod)로 분리해서 관리해도 좋을 것 같습니다!
3 participants
📄 PR 내용 요약
팀 수정하기, 삭제하기 구현
✅ 작업 내용 상세
📸 스크린샷 (선택사항)
💬 참고 사항