feat: dashbaorddetail- api 한개로 통합#2
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the dashboard data fetching logic by introducing a centralized detailTodos API and query, replacing individual goal detail fetches. It also updates query function imports and cleans up component styling. Feedback focuses on fixing broken character encoding in error messages, addressing potential data loss from client-side filtering of limited result sets, and reconsidering the removal of infinite scrolling in the task lists.
| ? 'GitHub Issue close�� �����߽��ϴ�. ��� �� �ٽ� �õ����ּ���.' | ||
| : 'GitHub PR merge�� �����߽��ϴ�. ��� �� �ٽ� �õ����ּ���.'; |
There was a problem hiding this comment.
한글 인코딩 문제로 인해 메시지가 깨져서 표시되고 있습니다. 사용자에게 올바른 안내 메시지가 전달되도록 수정이 필요합니다.
| ? 'GitHub Issue close�� �����߽��ϴ�. ��� �� �ٽ� �õ����ּ���.' | |
| : 'GitHub PR merge�� �����߽��ϴ�. ��� �� �ٽ� �õ����ּ���.'; | |
| ? 'GitHub Issue close에 실패했습니다. 잠시 후 다시 시도해주세요.' | |
| : 'GitHub PR merge에 실패했습니다. 잠시 후 다시 시도해주세요.'; |
| showToast(message, 'fail'); | ||
| } else { | ||
| showToast('할 일 상태 업데이트에 실패했습니다.', 'fail'); | ||
| showToast('���� ���� ������Ʈ�� �����߽��ϴ�.', 'fail'); |
There was a problem hiding this comment.
한글 인코딩 문제로 인해 토스트 메시지가 깨져서 표시되고 있습니다.
| showToast('���� ���� ������Ʈ�� �����߽��ϴ�.', 'fail'); | |
| showToast('할 일 상태 업데이트에 실패했습니다.', 'fail'); |
| showToast('���� ���� ������Ʈ�� �����߽��ϴ�.', 'fail'); | ||
| } | ||
| console.error('할 일 상태 업데이트 실패:', error); | ||
| console.error('���� ���� ������Ʈ ����:', error); |
There was a problem hiding this comment.
에러 로그의 한글 메시지가 깨져 있습니다.
| console.error('���� ���� ������Ʈ ����:', error); | |
| console.error('할 일 상태 업데이트 실패:', error); |
| fetchTodos.getTodos({ sort: 'LATEST', search: '', limit: 300, done: false }), | ||
| fetchTodos.getTodos({ sort: 'LATEST', search: '', limit: 300, done: true }), |
There was a problem hiding this comment.
전체 할 일 목록에서 상위 300개만 가져온 뒤 클라이언트 사이드에서 필터링하는 방식은 데이터 누락의 위험이 있습니다. 사용자의 전체 할 일 개수가 많을 경우, 특정 목표(특히 생성된 지 오래된 목표)에 속한 할 일이 이 300개 목록에 포함되지 않아 대시보드에서 해당 목표의 할 일이 비어 있는 것처럼 보일 수 있습니다. 각 목표별로 최신 할 일을 가져오거나, 서버에서 목표별로 그룹화된 데이터를 반환하는 전용 API를 사용하는 것을 권장합니다.
| defaultItems: TodoResponse[]; | ||
| searchItems: TodoResponse[] | null; |
There was a problem hiding this comment.
대시보드 API 통합 과정에서 ListBox의 무한 스크롤 기능(useInfiniteQuery)이 제거되었습니다. 현재는 API에서 전달받은 상위 10개의 할 일만 표시되며, 사용자가 더 많은 할 일을 확인하려면 검색 기능을 이용해야만 합니다. 대시보드에서 전체 할 일을 탐색하던 기존 사용자 경험을 유지하기 위해 '더 보기' 링크를 제공하거나 무한 스크롤 기능을 복구하는 것을 고려해 주세요.
1 participant
No description provided.