Skip to content
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

[FEAT] Task 타임블로킹 생성 api 연결 #217

Merged
merged 2 commits into from
Jul 18, 2024

Conversation

Kjiw0n
Copy link
Contributor

@Kjiw0n Kjiw0n commented Jul 18, 2024

작업 내용 🧑‍💻

  • Task 타임블로킹 생성 api 연결

알게된 점 🚀

기록하며 개발하기!

  • 10분이면 할 일을.. requestBody 변수명을 잘못 넣어서 시간을 많이 썼습니다. 명세를 잘 확인하자!

리뷰 요구사항 💬

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

스크린샷 2024-07-19 04 21 09

테스트 시 이 부분 참고해주세요!

관련 이슈

close #212

스크린샷 (선택)

Jul-19-2024 04-18-38

Copy link
Member

@seong-hui seong-hui left a comment

Choose a reason for hiding this comment

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

늦은 시간까지 고생이 많으십니다 ,, 엉엉 이벤트 클릭 후 시간을 긁으면 태스크 생김을 확인했습니다. 현재 서버에서 허용되지 않은 부분에 대해서 막아주고 있긴하지만 빠르게 프론트에서도 방지할 방법을 함께 고민해보도록 할게요 !!👍

@@ -23,7 +23,9 @@ function Today() {
// Task 목록 Get
const { data: stagingData, isError: isStagingError } = useGetTasks({ isTotal, sortOrder });
const { data: targetData, isError: isTargetError } = useGetTasks({ targetDate });
const { mutate } = useUpdateTaskStatus();
const { mutate, queryClient } = useUpdateTaskStatus();
Copy link
Member

Choose a reason for hiding this comment

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

queryClient를 사용하는 방법보다 mutate를 사용해서 요청을 하는 방식이 기존에 만들어둔 mutate함수를 사용할 수 있어서 좋을 것 같습니다! 해당 사항은 나중에 확인 해주세용 ~

@Kjiw0n Kjiw0n requested a review from lolarho July 18, 2024 19:55
Copy link

@lolarho lolarho left a comment

Choose a reason for hiding this comment

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

LGTM

@Kjiw0n Kjiw0n merged commit 53ff83b into develop Jul 18, 2024
2 checks passed
@wrryu09 wrryu09 deleted the feat/#212-timeblock-create-api branch July 22, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FEAT] Task TimeBlock 생성 post api 연결
3 participants