Skip to content

main에 merge - #696

Open
CallmeEterHist wants to merge 16 commits into
mainfrom
develop
Open

CallmeEterHist wants to merge 16 commits into
mainfrom
develop

Conversation

@CallmeEterHist

Copy link
Copy Markdown
Member

📋 PR 타입

해당하는 타입을 선택해 주세요 (복수 선택 가능)

  • feature — 새로운 기능 추가
  • fix — 버그 수정
  • style — 코드 스타일 수정 (로직 변경 없음)
  • refactor — 리팩토링
  • docs — 문서 수정
  • test — 테스트 코드
  • design — UI/UX 변경
  • chore — 설정/빌드 작업
  • merge

🗂 작업 영역

  • 워크벤치 — 회의 · 캡처 · AI검토 · 회의실
  • 업무 — 보드 · 액션 · 프로젝트 · 인수인계
  • 조직 — 역할별 대시보드 · 사원/회의실 관리 · 구독
  • 공유 셸 · 디자인 시스템 ⚠️ (셸 담당자만, 별도 PR)
  • 공통 · 인프라

🙋 관련 역할

  • OWNER (대표)
  • ADMIN (관리자)
  • LEADER (팀장)
  • MEMBER (사원)
  • SYSTEM (서비스 운영자 — 확장, 데모 제외)
  • 공통

📝 작업 내용

무엇을 · 왜 바꿨는지 작성해 주세요


✅ 작업 체크리스트

기본

  • 브랜치 명명 규칙 준수 (feature/meeting-capture#12 꼴, base develop)
  • 커밋 컨벤션 준수 (type: 제목 #{이슈번호})
  • 아래 Closes # 에 이슈 번호 기입
  • 불필요한 console · 주석 처리된 코드 제거
  • 민감 정보 없음 (토큰 · 키 · .env)

Z 필수

  • 🔐 권한 2축 검사 — 역할 가드 + 리소스 소유권(회의 담당자 등), 그리고 서버(Server Action/BFF)에서 재검사
  • 🧬 zod — 타입이 스키마에서 파생(z.infer) · 매퍼에 safeParse
  • 🕵️ 정직성 — 목 · 미구현 · 폴백이면 주석과 화면에 명시 (조용히 "되는 척" 금지)
  • 🎨 디자인 토큰 사용 (생 hex 하드코딩 없음) · 다크 값도 정의됨

품질

  • ⚡ 최적화 (이미지 next/image · 폰트 next/font · 무거운 것 next/dynamic · 시맨틱 태그)
  • ♿ a11y (label · role · alt · 키보드) · DnD면 키보드 대체 경로
  • ♻️ 재사용 확인 (components/ui → common → 도메인 순으로 먼저 확인)
  • 🧪 loading / error / empty 3상태
  • 브라우저 전용 API(STT · 녹음) 사용 시 미지원 · 권한거부 안내 있음

🔗 연관 이슈

Closes #


📸 스크린샷 (선택)

Before After

💬 리뷰 포인트

리뷰어가 중점적으로 봐야 할 부분


📝 추가 메모

hyun030514 and others added 16 commits August 21, 2026 23:08
[FIX] 보드 저장 실패 시 원인이 토스트 문구 하나로 가려지던 것 #676
[FIX] 회의·보드·내 액션·팀 액션·결제 화면의 목 인물/주소 하드코딩 제거 #678
[FIX] 인수인계서 관리 화면의 팀장/팀 하드코딩 제거 #680
[FIX] /app/my/actions에 OWNER 접근 가드 누락 (CLAUDE.md 원칙 위반) #682
[FIX] upload.test.ts 재시도 테스트가 병렬 실행 시 플레이키 #684
feat: 보드 드래그를 낙관적 UI + 즉시 저장으로 바꾼다 #686
test: handover 신청 lib 날짜 경계 회귀 방지 추가 #688
test: team-handover lib 3함수 회귀 방지 추가 #694
@coderabbitai

coderabbitai Bot commented Sep 14, 2026

Copy link
Copy Markdown

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: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: b9037011-ca52-413d-82fc-a41c3e90b434

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

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.

This branch has not been deployed

No deployments
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.

3 participants