Skip to content

feat(user): 리뷰 미디어 분리 제한 (사진 10 / 동영상 1)#78

Merged
chanwoo7 merged 1 commit intodevelopfrom
feat/mypage-figma-review-media-split
Apr 29, 2026
Merged

feat(user): 리뷰 미디어 분리 제한 (사진 10 / 동영상 1)#78
chanwoo7 merged 1 commit intodevelopfrom
feat/mypage-figma-review-media-split

Conversation

@chanwoo7
Copy link
Copy Markdown
Member

@chanwoo7 chanwoo7 commented Apr 29, 2026

Summary

  • 마이페이지 figma 05-review-write 명세 '사진 최대 10장 / 동영상 1개' 정책 반영
  • 기존 IMAGE/VIDEO 합산 10개 → 사진 10 + 동영상 1 (총 11개까지)
  • 에러 메시지를 사진/동영상 별도로 분리하여 사용자가 어느 쪽 제한에 걸렸는지 명확히 안내

변경 사항

  • MAX_IMAGE_COUNT(10) / MAX_VIDEO_COUNT(1) 상수 분리
  • validateMedia: type별 카운트로 변경
  • USER_REVIEW_ERRORS: TOO_MANY_MEDIA 제거 → TOO_MANY_IMAGES, TOO_MANY_VIDEOS 추가
  • 회귀 테스트 재작성 4건
    • 사진 10 + 동영상 1(총 11개) 통과
    • 사진 11장 reject
    • 동영상 2개 reject
    • 동영상 단독 1개 통과

Breaking 여부

  • 부분 Breaking
    • 기존: 사진 11개 첨부 시도 → 통과 가능 (기존 합산 10 정책에선 reject지만 본 PR로 인해 더 명확한 reject)
    • 새: 사진 10 + 동영상 1 = 11개 통과 (느슨해짐)
  • DB 데이터 0건 + 프론트 미개발 → 운영 영향 없음

Test plan

  • 로컬 yarn test:cov 통과 (840 tests)
  • CI check 통과
  • CI coverage-report 통과
  • CodeQL 통과

기존 정책은 IMAGE/VIDEO 합산 10개였으나, 마이페이지 figma 명세에 따라
사진 10장 / 동영상 1개로 분리한다 (총 11개까지 허용).

- MAX_IMAGE_COUNT(10) / MAX_VIDEO_COUNT(1) 상수 분리
- validateMedia: 단일 카운트에서 type별 카운트로 변경
- 에러 메시지 분리: TOO_MANY_MEDIA → TOO_MANY_IMAGES, TOO_MANY_VIDEOS
- 회귀 테스트 4건 (총 11개 통과 / 사진 11 reject / 동영상 2 reject / 동영상 단독 통과)
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: bebc1faa-be15-495a-88aa-d7e46e8616c4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/mypage-figma-review-media-split

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

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 96.6% 2812/2911
🟢 Branches 86.61% 1610/1859
🟢 Functions 93.09% 593/637
🟢 Lines 96.94% 2566/2647

Test suite run success

840 tests passing in 75 suites.

Report generated by 🧪jest coverage report action from e8c4717

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@chanwoo7 chanwoo7 merged commit 063910c into develop Apr 29, 2026
10 checks passed
@chanwoo7 chanwoo7 deleted the feat/mypage-figma-review-media-split branch April 29, 2026 16:10
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