Conversation
…g/codedang into t2567-create-problem-tab
…ataTable.tsx Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Code Review
This pull request refactors the problem list into a paginated table and introduces a "My Problems" grid view, adding generic table components, tab navigation, and enhanced pagination. Review feedback identified a logic error in the pagination data slicing, redundant navigation handlers in table rows, and potential routing issues in the "My Problems" section. Additionally, improvements were suggested for the loading skeletons to prevent layout shifts and a reminder was given to integrate real API data in place of current mocks.
…ataTable.tsx Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…mDataTable.tsx Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
|
✅ Syncing Preview App Succeeded Application: |
…ing/codedang into t2650-create-myproblem-tab
|
✅ Syncing Preview App Succeeded Application: |
|
작업하시느라 고생하셨습니다! 제가 찾은 리뷰거리 몇 개 공유해드려요~ |
| <IoFilter className="text-color-cool-neutral-30 mr-2 h-5 w-5" /> | ||
| State | ||
| </Button> | ||
| <SearchBar className="h-[46px] w-60" /> |
There was a problem hiding this comment.
searchBar들이 디자인과 달라서 종희님 searchBar component PR 머지되면 그에 맞게 수정해야 할 것 같아요!
There was a problem hiding this comment.
네, PR 머지 되면 그거까지 적용해서 다시 알려드릴게요!
|
✅ Syncing Preview App Succeeded Application: |
Description
https://www.figma.com/design/c7RefqPCB4HC9DuFhT7eFg/%EC%BD%94%EB%93%9C%EB%8B%B9-%EB%A6%AC%EB%94%94%EC%9E%90%EC%9D%B8--2025ver-?node-id=29387-14949&m=dev 피그마 링크

my problem 페이지 (데이터 없는 경우) /problem/my-problem/empty => 이 경로로 가면 볼 수 있고, 머지 전에 지우도록 하겠습니다.
published 탭 filter 기능이 어떻게 되는지 모르겠어서 일단 my problem 탭 작업부터 했습니다.
무한 스크롤 기능으로 구현되어 있습니다.
searchBar는 높이 구현이 안 되어서 컴포넌트 작업을 시작해야 할 거 같습니다.
데이터 없는 경우의 페이지도 작업 했습니다. 확인 부탁드려요! (경로 : /problem/my-problem/empty)
새 문제 생성 버튼은 누르면 /problem/create로 가도록 했고, filter는 버튼만 일단 만들어 뒀습니다.
Additional context
close TAS-2650
Before submitting the PR, please make sure you do the following
fixes #123).