Skip to content

Conversation

@wlrnjs
Copy link
Member

@wlrnjs wlrnjs commented Dec 29, 2025

Pull Request

관련 이슈

작업 내용

  • 일반 게시글 필터 UI 추가
  • 일반 게시글 목록 API 타입 위치 정리
  • 중복 타입 제거
  • 게시글 검색 관련 훅 제거
  • API 및 디자인 토큰이 누락된 부분은 주석으로 표시했습니다

참고 사항

  • PR 내용이 많아질 것 같아, 접근성 개선은 분리해서 올리도록 하겠습니다
  • 검색 관련 기능은 디자인 변경이 필요할 것 같아 작업하지 않았습니다.
  • FilterSection.tsx 파일 filter 컴포넌트 map 사용하지 않은 부분은
    map을 사용했을 때 오히려 코드 길이가 길어지고, 가독성이 저하되는 것 같아서 적용하지 않았습니다.
  • 필터를 적용했을때 아이콘 색상 변경은 추가하지 않았습니다.

체크리스트

  • 기능이 정상 동작하는지 확인
  • 로컬 빌드/스토리북/테스트 통과
  • 불필요한 코드/주석 제거

@wlrnjs wlrnjs self-assigned this Dec 29, 2025
@wlrnjs wlrnjs added ✨ feature 새로운 기능 추가, 요구사항 반영 🎨 style CSS, 레이아웃, 반응형, 퍼블리싱 관련 수정 (기능 변경 없음) D-3 3일 이내에 처리가 필요합니다. labels Dec 29, 2025
@vercel
Copy link

vercel bot commented Dec 29, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
fmi-fe Ready Ready Preview, Comment Jan 1, 2026 10:20pm

@github-actions
Copy link

github-actions bot commented Dec 29, 2025

Test Results

314 tests   314 ✅  26s ⏱️
 61 suites    0 💤
  1 files      0 ❌

Results for commit 7ba2e45.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@suhyeon0111 suhyeon0111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다! 코멘트 확인 부탁드릴게요

@wlrnjs wlrnjs merged commit 4ab8b13 into develop Jan 2, 2026
4 checks passed
@wlrnjs wlrnjs deleted the feat/list-filter-ui branch January 2, 2026 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

D-3 3일 이내에 처리가 필요합니다. ✨ feature 새로운 기능 추가, 요구사항 반영 🎨 style CSS, 레이아웃, 반응형, 퍼블리싱 관련 수정 (기능 변경 없음)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants