Feat/team PR

[hanjiwon1108]

💡 개요

team 모아보기 페이지 개발

📃 작업내용

리펙토링과 함께 모아보기에 맞는 type을 지정해서 조건문 작성
Profile과 Project를 모아서 만든 또 다른 type을 만들어서 검색 되게 작성

🔀 변경사항

📸 스크린샷

[insertjenkins]

🚀 배포 준비중

예상포트 : 4004

[Copilot]

Copilot reviewed 17 out of 18 changed files in this pull request and generated 1 comment.

Files not reviewed (1)

• package.json: Language not supported

Comments suppressed due to low confidence (1)

src/app/components/profile/Item.tsx:17

• [nitpick] The prop 'profile' is being used as an array in the mapping function; renaming it to 'profiles' could improve clarity regarding its contents.

profile.map((profile) => (

[Copilot]

The Filter component accepts a type that includes 'student' but does not define a query branch for it, which may lead to undefined behavior. Consider adding an appropriate branch or limiting the type union to only the supported cases.

[obtuse-triangle]

./src/app/components/project/filter/Filter.tsx:66:15
Type error: Property 'data' does not exist on type 'PostgrestResponseFailure | PostgrestResponseSuccess<{ created_at: string; email: string | null; isTeam: boolean; link: string[] | null; owner: string | null; profile_id: string; profile_name: string; }[]> | PostgrestResponseSuccess<...> | undefined'.

  64 |       }
  65 |
> 66 |       const { data, error } = await query;
     |               ^
  67 |
  68 |       if (error) {
  69 |         console.error('Error fetching search results:', error);

[kwonminjae5700]

대부분의 span 태그에 leading-[150%] tracking-normal 이 있는데 어떤 의도로 작성하신건가요?

[kwonminjae5700]

#0B50D0 색이 반복되어 tailwind.config.ts 에 정의해서 사용하는 것도 좋을 거 같습니다.

[kwonminjae5700]

모아보기가 있는 span 태그와 Filter, TotalList 를 하나의 컴포넌트로 만든 뒤 다른 동적인 부분들을 Layout 으로 만드는 것이 좋을 거 같습니다

[kwonminjae5700]

md:pr-16 md:pl-16 이 코드를 md:px-16 으로 사용할 수 있을 거 같습니다.

[kwonminjae5700]

Tag 컴포넌트에 24번째 라인에 tracking-normal 이랑 leading-[150%] 가 왜 필요한건가요?

[kwonminjae5700]

setSearch 보다는 다른 이름이 좋을 거 같습니다. 세터 함수 같습니다.

[kwonminjae5700]

14번째 줄에 Topics 라는 interface 가 대부분 Tag 에 쓰이던데 왜 이름은 Topics 인가요?

[kwonminjae5700]

TagList 에 Tag 가 생기면 화면이 움직입니다.

[kwonminjae5700]

왜 31번줄 css 에만 material-symbols-outlined 를 쓰셨나요?

[kwonminjae5700]

이렇게 되면 적용하기 버튼을 누르지 않아도 태그나 검색어가 적용될거 같습니다.
+
검색어에 한 번 문자를 넣을 때마다 supabase 에 요청을 보내는거 아닌가요? 이 코드는 꼭 수정이 필요해보입니다. React Query 를 쓰던지, project 와 team 데이터를 한 번 받아온 뒤 받아온 데이터를 가지고 filter 하는 방식으로 리팩토링 하는 것이 좋을 거 같습니다.

[kwonminjae5700]

38번줄에 console.log(data) 를 삭제해주셨으면 좋겠습니당