Skip to content

Comments

멘토링 삭제#381

Merged
HA-SEUNG-JEONG merged 1 commit intodevelopfrom
fix/study
Feb 6, 2026
Merged

멘토링 삭제#381
HA-SEUNG-JEONG merged 1 commit intodevelopfrom
fix/study

Conversation

@HA-SEUNG-JEONG
Copy link
Contributor

@HA-SEUNG-JEONG HA-SEUNG-JEONG commented Feb 6, 2026

🌱 연관된 이슈

☘️ 작업 내용

  • 그룹 스터디 내 드롭다운 필터에서 멘토링 삭제

🍀 참고사항

스크린샷

스크린샷 2026-02-06 19 59 54

Summary by CodeRabbit

릴리스 노트

  • 기능 변경
    • 학습 유형 필터에서 멘토링 옵션을 제거했습니다.

@HA-SEUNG-JEONG HA-SEUNG-JEONG self-assigned this Feb 6, 2026
@vercel
Copy link

vercel bot commented Feb 6, 2026

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

Project Deployment Actions Updated (UTC)
study-platform-client-dev Ready Ready Preview, Comment Feb 6, 2026 11:00am

@coderabbitai
Copy link

coderabbitai bot commented Feb 6, 2026

📝 Walkthrough

개요

study-filter.tsx 파일에서 MENTORING 옵션을 STUDY_TYPE_OPTIONS에서 제거하여, UI의 선택 가능한 학습 유형 목록에서 MENTORING을 삭제했습니다. 다른 로직이나 상태 관리는 변경되지 않았습니다.

변경사항

Cohort / File(s) 요약
필터링 옵션 제거
src/components/filtering/study-filter.tsx
MENTORING 학습 유형 옵션을 STUDY_TYPE_OPTIONS에서 제거했습니다.

예상 코드 리뷰 노력

🎯 1 (Trivial) | ⏱️ ~2분

관련 가능성이 있는 PR

  • 그룹스터디 필터링 로직 수정 #379: 두 PR 모두 UI 선택 로직에서 "MENTORING" 학습 유형 옵션을 제거합니다(메인 PR은 study-filter 옵션에서, 관련 PR은 그룹 스터디에서 필터링/제거).

🐰 한 줄의 선택지를 깔끔이 지우고
멘토링은 이제 안 보여요
필터는 더 명확하게 빛나고
사용자는 더 쉽게 고르지요
작지만 의미 있는 변화 ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 실제 변경사항과 일치합니다. STUDY_TYPE_OPTIONS에서 MENTORING 항목을 삭제한 변경사항을 '멘토링 삭제'라는 명확한 제목으로 요약했습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/study

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

@HA-SEUNG-JEONG HA-SEUNG-JEONG merged commit d8f8a39 into develop Feb 6, 2026
10 checks passed
@HA-SEUNG-JEONG HA-SEUNG-JEONG deleted the fix/study branch February 6, 2026 13:40
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