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 연결 #235

Merged
merged 9 commits into from
Jul 19, 2024
Merged

Conversation

jeeminyi
Copy link
Member

작업 내용 🧑‍💻

  • staging area와 target area에서 task 더블 클릭 시 상세 정보를 모달에 get 해오는 api를 연결했습니다.

알게된 점 🚀

기록하며 개발하기!

  • 으,,앙 ㅜㅜ 너무 많은 걸 알게 되었고, 기존에 제가 작업하던 api 연결과는 많이 다른 부분이 있어 어제 오늘 대가리 깨며 많이 배운 것 같습니다,, 사실 아직도 많이 이해가 필요하지만, 그래도 비교적 쉬운 get 부분을 처음으로 작업하면서 내려주기만 하면 되는 부분이 대부분이라 감 잡는데 정말 많은 도움이 되었습니다. 제가 get 부분에 온전히 집중할 수 있도록 다른 부분을 작업해준 승연이에게 고맙습니다... 타입 에러 잡아준 지원이와 많은 질문을 받아준 성희도,, 사실 제가 한건 없더요 ㅜㅜ
  • 옵셔널,, 을 많이 썼어요,, ㅋ
    • staging area에서는 targetDate가 없습니다. 요 부분이 좀 어려웠는데요, (왜냐면 staging area와 target area가 결국 modal이라는 같은 하위 컴포넌트를 공유하고 있기 때문에) staging area에서는 targetDate = null로 해주면서 해결하였습니다. 땡스 투 지원

리뷰 요구사항 💬

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

  • 원래는 time block에서도 상세 정보 모달이 노출되어야 하는데, 삭제 모달만 뜨는 것으로 수정되었습니다! 참고해주세요
  • targetDate 타입에러 때문에 애를 먹었는데요, 과정에서 유틸함수에 string을 추가해주었습니다. 괜찮은지 한번 봐주시면 감사하겠습니다!

관련 이슈

close #194

스크린샷

2024-07-19.5.39.30.mov

@Kjiw0n Kjiw0n force-pushed the feat/#194/task-description branch from ee03741 to 68e416a Compare July 19, 2024 09:08
Copy link
Contributor

@Kjiw0n Kjiw0n left a comment

Choose a reason for hiding this comment

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

잘 동작하는 것 확인했습니다!! 타입 오류 잡느라 수고하셨습니다 👍 👍

// Task 상세 설명 조회
const useTaskDescription = ({ taskId, targetDate, isOpen }: TaskDescriptionType) => {
const data = useQuery({
queryKey: ['today', 'taskDesc', taskId, targetDate],
Copy link
Contributor

Choose a reason for hiding this comment

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

쿼리키를 두개를 지정할 수 있었네요! 몰랐던 사실인데 배워갑니다 !!

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.

targetDate에서 타입에러 처리때문에 많이 고생하셨군요 ㅜ 아무래도 여러 곳에서 쓰이는 타입이다 보니 Date로도 쓰이고 string으로 쓰이는데 그 처리를 위해 잘 해주신 것 확인했습니다! 정말 고생 많으셨습니다 👍

@jeeminyi jeeminyi merged commit b837c2f into develop Jul 19, 2024
2 checks passed
@wrryu09 wrryu09 deleted the feat/#194/task-description 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 상세 조회 api 연결
3 participants