Skip to content

자료필터링 UI 기능 수정 : fix: build에러 해결 https://github.com/Sejong-Balsamic/M…#647

Merged
thswogh merged 1 commit intomainfrom
20251001_#643_기능개선_자료필터링_자료필터링_UI_기능_수정
Oct 1, 2025

Hidden character warning

The head ref may contain hidden characters: "20251001_#643_\uae30\ub2a5\uac1c\uc120_\uc790\ub8cc\ud544\ud130\ub9c1_\uc790\ub8cc\ud544\ud130\ub9c1_UI_\uae30\ub2a5_\uc218\uc815"
Merged

자료필터링 UI 기능 수정 : fix: build에러 해결 https://github.com/Sejong-Balsamic/M…#647
thswogh merged 1 commit intomainfrom
20251001_#643_기능개선_자료필터링_자료필터링_UI_기능_수정

Conversation

@thswogh
Copy link
Copy Markdown
Contributor

@thswogh thswogh commented Oct 1, 2025

Summary by CodeRabbit

  • 스타일
    • 문서 필터 하단시트의 고정 색상(파란색) 사용을 제거하고 기본 스타일을 적용했습니다. 대상: 인기 다운로드, 문서 요청, 티어별 문서 페이지.
    • 기능, 필터 동작, 트리거, 데이터 로딩 및 오류 처리에는 변화가 없습니다.
    • 화면 전반의 색상 일관성을 높이고 컴포넌트의 기본 테마를 준수합니다.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Oct 1, 2025

Walkthrough

세 곳의 페이지 컴포넌트에서 DocumentFilteringBottomSheet 사용 시 전달하던 activeColor prop을 제거했습니다. 기타 트리거, 필터링 상태, 콜백, 데이터 로딩 및 제어 흐름은 변경되지 않았습니다.

Changes

Cohort / File(s) Summary of changes
DocumentFilteringBottomSheet 색상 prop 제거
src/app/board/document/sub/hot-download/page.tsx, src/app/board/document/sub/request/page.tsx, src/app/board/document/tier/[postTier]/page.tsx
activeColor prop 전달 코드 제거. 컴포넌트는 기본 스타일 사용. 다른 props와 로직/흐름 변화 없음.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

파란색 물감은 주머니에 넣고,
시트는 본색 그대로 반짝-
클릭하면 열리고, 닫히면 고요,
토끼는 수염을 정리하며 끄덕!
“색은 비워도 흐름은 그대로.” 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title Check ⚠️ Warning 제목에 불필요한 GitHub 링크와 ‘build 에러 해결’라는 문구가 포함되어 있어 주요 변경사항인 DocumentFilteringBottomSheet 컴포넌트의 activeColor prop 제거를 명확히 전달하지 못하며, 지나치게 장황하고 구체성이 부족합니다. 제목을 간결하게 “DocumentFilteringBottomSheet에서 activeColor prop 제거”와 같이 변경하여 실제 변경사항을 분명히 드러내고 불필요한 링크ㆍ태그를 제거하세요.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 20251001_#643_기능개선_자료필터링_자료필터링_UI_기능_수정

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eaa37cc and 0415cde.

📒 Files selected for processing (3)
  • src/app/board/document/sub/hot-download/page.tsx (0 hunks)
  • src/app/board/document/sub/request/page.tsx (0 hunks)
  • src/app/board/document/tier/[postTier]/page.tsx (0 hunks)
💤 Files with no reviewable changes (3)
  • src/app/board/document/tier/[postTier]/page.tsx
  • src/app/board/document/sub/hot-download/page.tsx
  • src/app/board/document/sub/request/page.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Build

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 1, 2025

세종말싸미 빌드 결과: ✅ 빌드 성공

빌드 검증이 완료되었습니다.

@thswogh thswogh merged commit baf5cee into main Oct 1, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant