[REFACTOR] 목표/이슈 타입 및 변수명 재정의#60
Conversation
sunhwaaRj
changed the title
waldls
left a comment
waldls
left a comment
There was a problem hiding this comment.
모두 확인했습니다! 미리 common.ts 만들어주셔서 감사합니다 ㅎㅎ 수고하셨어용 👍🏻👍🏻
| // filter 변경마다 다른 데이터 선택 -> 추후 새로운 데이터 불러오도록 | ||
| const dummyGoalGroups = useMemo<GoalFilter[]>(() => { | ||
| switch (filter) { | ||
| case '상태': | ||
| return dummyStatusGoalGroups; | ||
| case '우선순위': | ||
| return dummyPriorityGoalGroups; | ||
| case '담당자': | ||
| return dummyManagerGoalGroups; | ||
| default: | ||
| return []; | ||
| } | ||
| }, [filter]); | ||
|
|
||
| const allGoalsFlat = dummyGoalGroups.flatMap((g) => g.goals); | ||
|
|
There was a problem hiding this comment.
P2: 현재 dummy...GoalGroups 데이터를 보면 정렬 순서가 하드코딩되어 있는 것 같은데,
향후 실제 API 연동 시에도 이 정렬 순서를 서버에서 보장해줄 예정인지,
아니면 프론트에서 정렬 로직(getSortedGrouped 등)을 계속 유지할 계획인지 궁금합니다!
비동기 함수는 useMemo에서 사용할 수 없는 것으로 알고 있어서, API 연동을 하게 된다면 useEffect나 React Query같은 비동기 처리 함수로 전환하게 될지 궁금해서요!
There was a problem hiding this comment.
상태 순서: 없음 → 진행중 → 해야할 일 → 완료 → 검토 → 삭제
우선순위 순서: 없음 → 긴급 → 높음 → 보통 → 낮음
해당 순서대로 백엔드에서 전달되고 있으며, 2차적으로 프론트에서 해당 순서로 오도록 처리하였습니다. 다른 담당자나 목표명 필터 시 없음이 제일 먼저, 이외는 가나다순으로 정렬됩니다.
더미데이터는 ui 테스트를 위해 넣어놓은 것이며, 말씀하신 대로 useEffect 나 쿼리 중 어떤 방식으로 연동 작업 진행할 지는 추후 회의에서 얘기하고 통일해서 진행하면 될 것 같아요!!
There was a problem hiding this comment.
네넵 답변 감사합니다! UI 구현인데 추후 API 연동이 어떤식으로 이루어질지 궁금해서 여쭤봤습니다..ㅎㅎ
코드 너무 깔끔하게 짜주신 것 같아요 수고하셨습니다! 😍
4 participants
체크리스트
📌 관련 이슈번호
✅ Key Changes
📸 스크린샷 or 실행영상
51.mp4
💬 To Reviewers
common.ts에 1차적으로 api 구조에 맞는 respose type 작성하였습니다. 추후 해당 파일 및 폴더에 각자 담당한 부분 작성하면 됩니다.