Skip to content

[HOTFIX] 활동보고서 term 7로 수정#400

Merged
ujinsim merged 1 commit into
mainfrom
hotfix/#399
Jun 8, 2026
Merged

[HOTFIX] 활동보고서 term 7로 수정#400
ujinsim merged 1 commit into
mainfrom
hotfix/#399

Conversation

@ujinsim

@ujinsim ujinsim commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

🔥 연관 이슈

🚀 작업 내용

🤔 고민했던 내용

💬 리뷰 중점사항

Summary by CodeRabbit

  • Refactor
    • 보고서 항목 조회 로직이 변경되었습니다. 이제 고정된 기간을 기준으로 계산되며, 동적 조회 메커니즘을 사용하지 않습니다.

@vercel

vercel Bot commented Jun 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ddingdong-fe Ready Ready Preview, Comment Jun 8, 2026 8:51am

@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: c56fbf62-325a-44d1-8069-162488f5c044

📥 Commits

Reviewing files that changed from the base of the PR and between fc1ff93 and 767b9d4.

📒 Files selected for processing (1)
  • apps/web/app/admin/report/_hooks/useTerm.ts

변경 사항 요약

활동보고서 전용 useTerm 훅에서 원격 API를 통한 현재 항목(term) 동적 조회 로직을 제거하고, 항목 값을 고정된 숫자 7로 변경했습니다.

변경 사항

활동보고서 term 설정 변경

계층 / 파일 요약
현재 항목 쿼리 제거 및 고정값 적용
apps/web/app/admin/report/_hooks/useTerm.ts
원격 조회 useSuspenseQuery(reportQueryOptions.currentTerm())를 제거하고, currentTerm{ term: 7 }로 하드코딩. isClosed, isFuture, currentTerm 계산 시 동적 조회 값 대신 이 고정된 term 값을 기준으로 동작

예상 코드 리뷰 범위

🎯 3 (중간) | ⏱️ ~20분

제안된 레이블

fix

제안된 리뷰어

  • JangYEhoon00
  • keemsebin
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive PR 설명이 템플릿 플레이스홀더만 포함되어 있으며, 실제 구현 내용이나 리뷰 중점사항이 작성되지 않았습니다. 작업 내용, 고민했던 내용, 리뷰 중점사항 섹션을 채워서 변경사항의 영향도와 이유를 명확히 해주세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 주요 변경사항을 명확하게 설명합니다. 'useTerm' 훅에서 term 값을 7로 변경하는 핫픽스 내용과 일치합니다.
Linked Issues check ✅ Passed PR에서 useTerm 훅의 currentTerm.term 값을 7로 변경했으며, 이는 연관 이슈 #399의 '활동보고서 term 수정' 요구사항을 충족합니다.
Out of Scope Changes check ✅ Passed PR의 모든 변경사항(+1/-7 라인)이 활동보고서 term을 7로 수정하는 목표에 직접 관련되어 있으며, 범위 외의 변경이 없습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix/#399

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 github-actions Bot added the hotfix 신속한 처리가 필요한 이슈 label Jun 8, 2026
@ujinsim ujinsim merged commit 44b7da3 into main Jun 8, 2026
9 checks passed
@ujinsim ujinsim deleted the hotfix/#399 branch June 8, 2026 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hotfix 신속한 처리가 필요한 이슈

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[HOTFIX] 활동보고서 term 수정

1 participant