-
Notifications
You must be signed in to change notification settings - Fork 4
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
[토너먼트] 강제종료 #508
[토너먼트] 강제종료 #508
Conversation
|
@@ -15,12 +15,12 @@ const TournamentDeleteButton = ({ onClick }: TournamentStartButtonProps) => { | |||
backgroundColor: '#FF2176', | |||
border: 'none', | |||
}} | |||
onClick={onClick} | |||
onClick={() => onClick(roomId)} // props로 받은 roomId를 사용 |
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 async function deleteRoom(roomId: number) { | ||
await post(`/gift/tournament-end/${roomId}`); | ||
} |
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.
오 delete는 한 번도 안해봤는데 post만 보내도 되는군요! 하나 또 배워갑니다 ㅎㅎ
export const useDeleteRoom = ({ roomId }: { roomId: number }) => { | ||
const queryClient = useQueryClient(); | ||
|
||
const mutation: any = useMutation({ |
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.
혹시 mutation을 {mutation}으로 받아도 타입 설정을 해줘야할까요??
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.
되게 고통받으셨는데 결국 해결하셨군요!! 고생많으셨습니다. mutation 부분에 리뷰하나 남겼으니 확인 부탁드릴게요. 항상 감사합니다 ㅎㅎ
이슈 넘버
구현 사항
Need Review
📸 스크린샷
Reference