Skip to content

Comments

feat: 에러 코드 추가 및 사전 의견 버튼 disabled 처리#98

Open
mgYang53 wants to merge 1 commit intodevelopfrom
feat/error-and-main-meeting-97
Open

feat: 에러 코드 추가 및 사전 의견 버튼 disabled 처리#98
mgYang53 wants to merge 1 commit intodevelopfrom
feat/error-and-main-meeting-97

Conversation

@mgYang53
Copy link
Contributor

@mgYang53 mgYang53 commented Feb 23, 2026

🚀 풀 리퀘스트 제안

📋 작업 내용

최신 API 스펙에 추가된 에러 코드 5개를 반영하고, 메인페이지 예정 약속의 "사전 의견 작성하기" 버튼에 대해 템플릿 미확정 시 disabled 처리를 구현했습니다.

🔧 변경 사항

  • src/api/errors.ts: 신규 에러 코드 5개 추가 (E000, M016, M017, R106, R108)
  • src/features/meetings/meetings.types.ts: MyMeetingListItempreOpinionTemplateConfirmed 필드 추가
  • src/pages/Home/components/HomeMeetingCard.tsx: 사전 의견 템플릿 미확정 시 버튼 disabled 처리 및 TODO 주석 제거

📸 스크린샷 (선택 사항)

N/A

📄 기타

관련 이슈: #97

Summary by CodeRabbit

  • 버그 수정

    • 서버 오류, 회의 참여/업데이트 만료 시간, AI 요약 미발견, 회고 중복 생성 등 신규 오류 메시지 추가
  • 새로운 기능

    • 사전 의견 템플릿 확인 상태 기반 회의 작업 흐름 개선
    • 템플릿 확인 전까지 사전 의견 작성 버튼 비활성화

신규 에러 코드 5개(E000, M016, M017, R106, R108) 추가.
MyMeetingListItem에 preOpinionTemplateConfirmed 필드 추가 후,
메인페이지 예정 약속의 사전 의견 작성 버튼 disabled 처리 반영.
@mgYang53 mgYang53 linked an issue Feb 23, 2026 that may be closed by this pull request
5 tasks
@mgYang53 mgYang53 self-assigned this Feb 23, 2026
@mgYang53 mgYang53 added feat 새로운 기능 추가 fix 기존 기능 수정 labels Feb 23, 2026
@mgYang53 mgYang53 changed the title feat: 에러 코드 추가 및 사전 의견 버튼 disabled 처리 (#97) feat: 에러 코드 추가 및 사전 의견 버튼 disabled 처리 Feb 23, 2026
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/api/errors.ts`:
- Around line 133-134: There is a missing R107 error code between
AI_SUMMARY_NOT_FOUND and RETROSPECTIVE_ALREADY_CREATED; either add a new export
entry for R107 (matching the backend API spec) with the proper identifier and
value 'R107' in the same object where AI_SUMMARY_NOT_FOUND and
RETROSPECTIVE_ALREADY_CREATED are defined, or if R107 was intentionally skipped,
rename RETROSPECTIVE_ALREADY_CREATED to use 'R107' (and update any references)
or add an inline comment explaining the intentional gap so clients won't
mis-handle unexpected codes; update the object containing AI_SUMMARY_NOT_FOUND
and RETROSPECTIVE_ALREADY_CREATED accordingly.

ℹ️ Review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5ec4e8f and 1b75cf1.

📒 Files selected for processing (3)
  • src/api/errors.ts
  • src/features/meetings/meetings.types.ts
  • src/pages/Home/components/HomeMeetingCard.tsx

@fc-de fc-de deleted a comment from coderabbitai bot Feb 23, 2026
@fc-de fc-de deleted a comment from coderabbitai bot Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 새로운 기능 추가 fix 기존 기능 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] 에러 코드 추가 및 사전 의견 작성 버튼 disabled 처리

2 participants