Skip to content

[Bugfix/#351] 타임라인 performanceStatus ABOVE_AVERAGE → ABOVE_AVG로 값명 변경 - #354

Merged
jjjsun merged 1 commit into
developfrom
bugfix/#351
Jul 30, 2026
Merged

[Bugfix/#351] 타임라인 performanceStatus ABOVE_AVERAGE → ABOVE_AVG로 값명 변경#354
jjjsun merged 1 commit into
developfrom
bugfix/#351

Conversation

@jjjsun

@jjjsun jjjsun commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

🚨 관련 이슈

Closed #351

✨ 변경사항

  • 🐞 BugFix Something isn't working
  • 💻 CrossBrowsing Browser compatibility
  • 🌏 Deploy Deploy
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (storybook, jest, etc.)

✏️ 작업 내용

타임라인 performaceStatusABOVE_AVERAGEABOVE_AVG로 백엔드와 통일

😅 미완성 작업

N/A

📢 논의 사항 및 참고 사항

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

Summary by CodeRabbit

  • 개선 사항
    • 타임라인 성과 상태 명칭을 ABOVE_AVERAGE에서 ABOVE_AVG로 통일했습니다.
    • 주간·일간·월간 타임라인과 성과 요약 영역에 변경된 상태값을 반영했습니다.
    • 해당 상태에 대한 범례와 UI 스타일 표시가 정상적으로 유지됩니다.

@jjjsun
jjjsun requested review from Seojegyeong and YermIm July 30, 2026 07:54
@jjjsun jjjsun self-assigned this Jul 30, 2026
@jjjsun jjjsun added the 🐞 BugFix Something isn't working label Jul 30, 2026
@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

타임라인 성과 상태 문자열을 ABOVE_AVERAGE에서 ABOVE_AVG로 변경했습니다. 상태 타입, 범례 순서, 스타일 매핑과 주간·일간·월간 및 요약 패널 mock 데이터가 함께 갱신되었습니다.

Changes

타임라인 성과 상태 변경

Layer / File(s) Summary
성과 상태 계약 및 스타일 매핑
src/types/timeline/api.ts, src/constants/timeline/statusStyle.ts
성과 상태 상수와 파생 타입, 범례 순서 및 스타일 매핑 키를 ABOVE_AVG로 변경했습니다.
타임라인 mock 상태 데이터 갱신
src/types/timeline/timeline.mock.ts
주간·일간·월간 타임라인과 성과 요약 패널 mock의 performanceStatus 값을 갱신했습니다.

Estimated code review effort: 2 (Simple) | ~5 minutes

Possibly related PRs

Suggested reviewers: seojegyeong, yermim

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed #351의 핵심 요구사항인 ABOVE_AVERAGE를 ABOVE_AVG로 통일하는 변경이 상수, 타입, mock까지 반영되어 있습니다.
Out of Scope Changes check ✅ Passed 변경 범위가 타임라인 performanceStatus 명칭 정정에만 집중되어 있어 별도 범위 이탈은 보이지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed 타임라인 performanceStatus 값을 ABOVE_AVERAGE에서 ABOVE_AVG로 바꾸는 핵심 변경을 정확히 요약했습니다.
Description check ✅ Passed 관련 이슈, 변경사항, 작업 내용, 미완성 작업이 모두 있어 템플릿 요구를 대부분 충족합니다.
✨ 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 bugfix/#351

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.

@github-actions

Copy link
Copy Markdown

📚 Storybook 배포 완료

항목 링크
📖 Storybook https://69a147b60a56365d9e2185ef-dfrlftpzkr.chromatic.com/
🔍 Chromatic https://www.chromatic.com/build?appId=69a147b60a56365d9e2185ef&number=489

@jjjsun jjjsun changed the title [Bugfix/#351] 타임라인 performanceStatus ABOVE_AVERAGE → ABOVE_AVG [Bugfix/#351] 타임라인 performanceStatus ABOVE_AVERAGE → ABOVE_AVG로 값명 변경 Jul 30, 2026

@Seojegyeong Seojegyeong left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

P4: 확인했습니다

@jjjsun
jjjsun merged commit 309dd2b into develop Jul 30, 2026
3 checks passed
@jjjsun
jjjsun deleted the bugfix/#351 branch July 30, 2026 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 BugFix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐞 [BugFix] 타임라인 performanceStatus 값 이름 수정

2 participants