Skip to content

[FIX] 액션 분배 확정 시 필수 필드 강조 · README 최신화 #673 - #674

Merged
hyun030514 merged 2 commits into
developfrom
fix/review-required-field-highlight-and-readme#673
Aug 20, 2026
Merged

hyun030514 merged 2 commits into
developfrom
fix/review-required-field-highlight-and-readme#673

Conversation

@zana0422-creator

Copy link
Copy Markdown
Contributor

📋 PR 타입

  • fix — 버그 수정
  • docs — 문서 수정

🗂 작업 영역

  • 워크벤치 — 회의 · 캡처 · AI검토 · 회의실
  • 공통 · 인프라 (문서)

🙋 관련 역할

  • 전 역할 공통

📝 작업 내용

1) 액션 분배 확정 시 필수 필드 강조

  • 확정 버튼 disabled 대신 눌러 보면 즉시 강조. 빈 제목·세부·마감 필드는 빨간 테두리 + 빨간 안내 텍스트로 승격
  • `InlineEditableField`에 `isInvalid` prop 배선 — Textarea·input·button 세 상태 모두 대응
  • `DatePickerField` 마감일에 `aria-invalid` + 조건부 `border-destructive`
  • 반려된 행은 강조 대상 제외 (반려는 이미 사람이 판단한 상태 · 확정 요청에 안 실림)
  • 안내 문구도 회색 → 빨강으로 승격 · "빨갛게 표시된 자리를 채워 주세요"

2) README 최신화

  • 라우트 그룹 최신 이름 반영 — `(role)` → `(shell)/(authority)`, `(gate)`·`(dev)`·`(system)` 추가
  • features 폴더 구조 정정 — `schemas` → `mapper` (실제 사용)
  • 27개 도메인 규모, Server Action 22곳, next/dynamic 14곳, Jest 1,555개 통과 등 실측 수치
  • `hooks/`·`mocks/`·`styles/`·`types/` 폴더 누락 보완
  • 최적화 축(three.js · MarkdownEditor · Calendar · Recharts · STT 등 실제 지연 로드 지점) 언급

✅ 작업 체크리스트

  • 브랜치 명명 규칙 준수 (base: develop)
  • 커밋 컨벤션 준수
  • 아래 `Closes #` 에 이슈 번호 기입
  • 불필요한 `console` · 주석 코드 없음
  • 민감 정보 없음
  • typecheck 통과
  • jest 1,555개 통과 (review 도메인 13개 재확인)

🔗 연관 이슈

Closes #673

💬 리뷰 포인트

  • `inline-editable-field.tsx` — Textarea·input·button 세 상태의 `isInvalid` 처리가 일관되는지
  • `meeting-review-view.tsx` — 확정 버튼 클릭 로직: `hasMissingRequired`면 모달 안 열고 강조만 승격
  • README의 실측 수치가 실제 코드와 일치하는지 (임의 값 없음 · 전부 코드 훑어 확인)

- 확정 버튼 disabled 걷어내고, 눌러 보면 비어 있는 필드가 빨간 테두리로 승격
- InlineEditableField에 isInvalid 배선(제목·세부 내용) · DatePickerField 마감일에 aria-invalid
- 반려된 행은 강조 대상 제외 (반려는 이미 사람이 판단한 상태)
- README: 라우트 그룹 최신 이름 · 27개 도메인 · Server Action 22개 · Jest 1,555개 통과 반영
@coderabbitai

coderabbitai Bot commented Aug 19, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7e90ad18-8722-4fb5-9627-258f54c1e581


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.

@hyun030514 hyun030514 added this to the [M4] 발표 준비 milestone Aug 19, 2026
@hyun030514 hyun030514 changed the title fix: 액션 분배 확정 시 필수 필드 강조 · README 최신화 #673 [FIX] 액션 분배 확정 시 필수 필드 강조 · README 최신화 #673 Aug 19, 2026

@hyun030514 hyun030514 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다.

@hyun030514
hyun030514 merged commit b1ed34a into develop Aug 20, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Z Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[FIX] 액션 분배 확정 시 필수 필드 강조 + README 최신화

2 participants