Skip to content

Fix/peer review main#449

Merged
HA-SEUNG-JEONG merged 2 commits into
mainfrom
fix/peer-review-main
Mar 25, 2026
Merged

Fix/peer review main#449
HA-SEUNG-JEONG merged 2 commits into
mainfrom
fix/peer-review-main

Conversation

@HA-SEUNG-JEONG

@HA-SEUNG-JEONG HA-SEUNG-JEONG commented Mar 25, 2026

Copy link
Copy Markdown
Contributor

🌱 연관된 이슈

☘️ 작업 내용

🍀 참고사항

스크린샷 (선택)

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 외부 링크의 자동 검증 및 포맷팅 기능 추가로 링크 입력 시 일관성 있는 처리 제공
  • 버그 수정

    • 숙제 및 스터디 자료의 첨부 링크 유효성 검사 강화
    • 잘못된 링크 형식 자동 수정 (프로토콜 추가 등)
  • 개선 사항

    • 숙제 작성 시 텍스트 입력 최대 글자 수를 5,000에서 1,000으로 조정

HA-SEUNG-JEONG and others added 2 commits March 25, 2026 16:49
- 과제 제출 과정에서 링크 입력 유효성 추가 검증
링크 유효성 수정(cb309b4) cherry-pick 과정에서
함께 들어온 className 토큰 변경을 이전 상태로 복원.
formatExternalLink / isValidUrl 링크 관련 변경은 유지.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@HA-SEUNG-JEONG HA-SEUNG-JEONG self-assigned this Mar 25, 2026
@vercel

vercel Bot commented Mar 25, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
study-platform-client-dev Error Error Mar 25, 2026 10:58am

@coderabbitai

coderabbitai Bot commented Mar 25, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

외부 링크 유효성 검사 및 정규화를 위한 새로운 유틸리티 함수(isValidUrl, formatExternalLink)를 추가하고, 이를 숙제 제출/수정 모달 및 여러 컴포넌트에 적용하여 URL 입력을 일관되게 검증하고 포맷팅합니다.

Changes

Cohort / File(s) Summary
유틸리티 함수 추가
src/utils/validation.ts, src/utils/format.ts
isValidUrl 함수로 URL 유효성 검사(공백 제거, 점 포함 여부, 프로토콜 처리)와 formatExternalLink 함수로 URL 정규화(프로토콜 자동 추가, 공백 제거) 구현
숙제 모달 업데이트
src/components/common/modals/edit-homework-modal.tsx, src/components/common/modals/submit-homework-modal.tsx
attachmentLink 필드에 대해 isValidUrl을 사용한 Zod 검증 추가 및 제출 시 formatExternalLink로 링크 정규화; 텍스트 최대 길이 5000에서 1000으로 조정
컴포넌트 링크 포맷팅
src/components/contents/homework-detail-content.tsx, src/components/hall-of-fame/mvp-team-card.tsx, src/components/lists/study-list-table.tsx, src/components/schedule/today-study-card.tsx, src/features/mentoring/ui/note-consultation/note-consultation-composite.tsx, src/features/study/one-to-one/history/ui/study-history-row.tsx
href 속성에 formatExternalLink 적용하여 모든 외부 링크 일관되게 정규화; 유효하지 않은 링크 렌더링 방지

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 외부 링크를 다듬어, 깔끔하게 정렬하고,
URL의 옷매무새를 맞춰주는 형식지기,
검증으로 안전하게, 정규화로 일관되게,
여섯 컴포넌트가 손잡고 춤을 춥니다. 🔗✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning 제목 'Fix/peer review main'은 변경 내용의 실제 핵심 주제(외부 링크 검증 및 포맷팅)와 일치하지 않으며, 변경사항의 구체적인 내용을 명확하게 전달하지 못합니다. 제목을 'Format external links and add URL validation for homework submissions'와 같이 변경하여 실제 변경 내용을 정확하게 반영하도록 개선하세요.
Docstring Coverage ⚠️ Warning Docstring coverage is 37.50% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/peer-review-main

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.

@HA-SEUNG-JEONG HA-SEUNG-JEONG merged commit a9fe781 into main Mar 25, 2026
7 of 9 checks passed
@HA-SEUNG-JEONG HA-SEUNG-JEONG deleted the fix/peer-review-main branch March 25, 2026 07:59
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