xction-dev / xction.co.kr

Xction!의 홈페이지를 만들고 있습니다
0 stars 0 forks source link

Feat/add watch types #17

Closed nuagenic closed 8 months ago

nuagenic commented 8 months ago

작업 내용

이슈 및 궁금증

  1. 우선 SampleUserService처럼 queryFn을 따로 작성하여 import하고자 했으나, 계속 에러가 나서(type error로 추정됨. No overload matches this call) 깡으로 가져다 썼습니다. 다이나믹 라우팅 처리된 api를 fetch해서 일어나는 문제일 수 있을 거 같은데 봐주시면 감사하겠습니다
  2. 클라이언트가 접속한 페이지의 id 번호를 가져와, 그것으로 GET 및 POST 요청을 보내야 하는 것으로 이해했습니다. 그래서 useSampleProjectServiceid 파라미터를 주고, pages 작업할 때 넣어주는 방식으로 작업해보았습니다. 그런데 문제는.. 가령 영상이 끝난 다음에 POST 요청을 보내고 싶은데, 이것은 components/Player.tsx에서 finishWatchingProject 함수를 콜하고 싶단 말이죠? 그런데 일단 이 훅을 component 에서 불러도 되는지, 부르는 게 맞는지? 그런데 만약 그렇게 부르는 게 맞다면, component 에는 id 값이 없으니까, 파라미터에 값을 채워줄 수가 없잖아요? 그러면 id의 types를 number | null 이런 식으로 처리하는 건지? 굉장히 골치 아팠습니다. 그래서 POST는 일단 보류. 적합한 방식 알려주시면 감사하겠습니다.
  3. Project는 User와 달리 requestbodyDTO가 없던데, 그럼 POST 시 body type은 any로 처리하면 되는 걸까요?
  4. 리베이스 머지 제대로 한 거 맞나요? 원래 이렇게 PR 걸면 diff가 준영 작업분까지 다 뜨는 게 맞나??
nuagenic commented 8 months ago

코멘트 주신 부분은 다 반영했으며, POST 관련해서는 얘기할 게 아직 있는데 내일 회의 때 해도 될 것 같습니다! 일단 머지하고 다음 PR에서 보완할 사항 있으면 보완하겠습니다~