Skip to content

[FIX] 제보 등록 화면 QA 대응#81

Merged
taipaise merged 1 commit intodevelopfrom
fix/report
Feb 7, 2026
Merged

[FIX] 제보 등록 화면 QA 대응#81
taipaise merged 1 commit intodevelopfrom
fix/report

Conversation

@taipaise
Copy link
Collaborator

@taipaise taipaise commented Feb 7, 2026

🌁 Background

  • 제보하기 등록화면 QA 대응을 진행했습니다

👩‍💻 Contents

Summary by CodeRabbit

변경 사항

  • 새 기능
    • 텍스트 입력 필드에 최대 길이 제한 기능 추가 (콘텐츠 입력 시 최대 150자)
    • 키보드 표시/숨김 시 화면 자동 조정 기능 추가
    • 사진 선택 시 최대 개수 제한 및 초과 시 알림 기능 추가

@taipaise taipaise self-assigned this Feb 7, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 7, 2026

🌟 Walkthrough

리포트 텍스트 입력 시 최대 길이 제한을 추가하고, 키보드 가시성 이벤트를 처리하며, 사진 선택 제한 로직을 구현했습니다. 또한 선택된 사진 개수를 뷰 모델에서 노출하는 계산 속성을 추가했습니다.

📋 Changes

Cohort / File(s) Summary
텍스트 길이 제한
ReportTextView.swift
선택적 maxLength 매개변수를 추가하고, 초기화 함수 서명을 업데이트했습니다. textViewDidChange와 shouldChangeTextIn 델리게이트 메서드에서 텍스트 길이를 검증합니다. isFirstResponder 오버라이드도 추가되었습니다.
키보드 처리 및 사진 선택 로직
ReportRegistrationViewController.swift
ReportTextView에 maxLength: 150을 설정하여 초기화합니다. 키보드 표시/숨김 옵저버를 등록하고 스크롤 뷰 인셋을 조정합니다. 사진 피커의 selectionLimit 계산에 선택 가능한 사진 개수 확인 로직을 추가했습니다.
뷰 모델 확장
ReportRegistrationViewModel.swift
selectedPhotoSubject의 항목 개수를 반환하는 selectedPhotoCount 계산 속성을 추가했습니다.

⏱️ Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

🐰 Poem

텍스트 길이를 제한하고,
키보드 춤을 추며,
사진 개수를 세어주는
영리한 토끼의 선물 🥕✨
입력의 경계, 이제 깔끔하도다!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 제보 등록 화면의 QA 대응이라는 핵심 변경사항을 명확하게 요약하고 있으며, 실제 코드 변경(텍스트 길이 제한, 키보드 처리, 사진 선택 제한)과 완전히 일치합니다.

✏️ 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/report

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.

@taipaise taipaise merged commit de8a12d into develop Feb 7, 2026
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