Skip to content

feat: 전적 테이블 이벤트 열 제거·중앙정렬·날짜 로케일 포맷 적용#32

Merged
umsungjun merged 1 commit into
mainfrom
feat/fight-record-table-cleanup
Jul 22, 2026
Merged

feat: 전적 테이블 이벤트 열 제거·중앙정렬·날짜 로케일 포맷 적용#32
umsungjun merged 1 commit into
mainfrom
feat/fight-record-table-cleanup

Conversation

@umsungjun

@umsungjun umsungjun commented Jul 22, 2026

Copy link
Copy Markdown
Owner
  • 크롤러가 event를 빈 값으로 저장해 항상 비어 나오던 이벤트 열 제거함
  • 테이블 헤더·셀 전체 중앙정렬 처리함
  • date-utils에 formatFullDate 추가해 날짜를 로케일별 표기(2025년 11월 1일 / Nov 1, 2025)로 변환함
  • 미사용 fightRecord.event 번역 키 정리함

Summary by CodeRabbit

  • 개선 사항
    • 경기 기록 표의 열 순서를 결과, 상대 선수, 경기 방식, 라운드, 날짜로 정리했습니다.
    • 날짜가 한국어와 영어 등 현재 지역 설정에 맞는 전체 날짜 형식으로 표시됩니다.
    • 경기 기록 표의 정렬과 셀 표시를 개선했습니다.

- 크롤러가 event를 빈 값으로 저장해 항상 비어 나오던 이벤트 열 제거함
- 테이블 헤더·셀 전체 중앙정렬 처리함
- date-utils에 formatFullDate 추가해 날짜를 로케일별 표기(2025년 11월 1일 / Nov 1, 2025)로 변환함
- 미사용 fightRecord.event 번역 키 정리함
Copilot AI review requested due to automatic review settings July 22, 2026 12:13
@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
lets-ko Building Building Preview, Comment Jul 22, 2026 12:13pm

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: bbf59737-4741-4a03-a0d1-3c9b4a2c8c41

📥 Commits

Reviewing files that changed from the base of the PR and between 75bbde4 and a817873.

📒 Files selected for processing (4)
  • src/components/fighter/FightRecord.tsx
  • src/lib/date-utils.ts
  • src/messages/en.json
  • src/messages/ko.json

📝 Walkthrough

Walkthrough

FightRecord가 로케일 기반 전체 날짜 포맷을 사용하도록 변경되고, event 열과 번역 키가 제거되었습니다. 나머지 열의 순서와 정렬, 셀 구성이 새 헤더에 맞게 조정되었습니다.

Changes

Fight record display updates

Layer / File(s) Summary
전체 날짜 포맷 유틸리티
src/lib/date-utils.ts
YYYY-MM-DD 입력을 KST 기준으로 로케일 포맷하고, 빈 값과 포맷 실패 시 폴백을 처리하는 formatFullDate를 추가했습니다.
FightRecord 테이블 통합
src/components/fighter/FightRecord.tsx, src/messages/en.json, src/messages/ko.json
로케일 훅과 날짜 포맷을 적용하고, event 열을 제거했으며, 나머지 열의 순서·정렬·셀 구성을 변경했습니다.

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

Possibly related PRs

  • umsungjun/lets-ko#28: date-utils.ts의 KST 기반 날짜 포맷 기능과 관련된 변경입니다.

Suggested reviewers: copilot

Poem

당근 먹은 토끼가 날짜를 봐요
로케일 따라 예쁘게 읽어요
이벤트 열은 살짝 뛰어넘고
새 순서 표에 발맞춰 총총
깔끔한 기록, 깡충!

✨ 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 feat/fight-record-table-cleanup

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install timed out. The project may have too many dependencies for the sandbox.


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.

@umsungjun
umsungjun merged commit d86d032 into main Jul 22, 2026
2 of 4 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

전적(Fight Record) 테이블에서 항상 비어 있던 event 열을 제거하고, 날짜 표기를 로케일(ko/en)에 맞춰 “연도 포함 전체 날짜” 형식으로 통일해 UI 품질과 일관성을 개선하는 PR입니다.

Changes:

  • 전적 테이블의 event 열(헤더/셀) 제거 및 테이블 헤더·셀 중앙 정렬 적용
  • date-utilsformatFullDate() 추가로 "YYYY-MM-DD" 캘린더 날짜를 로케일별 전체 날짜로 포맷
  • 더 이상 사용하지 않는 fightRecord.event 번역 키(ko/en) 제거

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/messages/ko.json fightRecord.event 번역 키 제거로 미사용 i18n 정리
src/messages/en.json fightRecord.event 번역 키 제거로 미사용 i18n 정리
src/lib/date-utils.ts 전적 등 과거 날짜 표기를 위한 formatFullDate() 유틸 추가
src/components/fighter/FightRecord.tsx 이벤트 열 제거, 중앙 정렬 적용, 날짜를 formatFullDate()로 로케일 포맷 처리

Comment thread src/lib/date-utils.ts
Comment on lines +58 to +60
* @param dateStr - "YYYY-MM-DD" 형식의 날짜 (그 외 형식은 Date 파싱 시도)
* @param locale - "ko" | "en"
* @returns 연도 포함 날짜 문자열, 파싱 실패 시 원본 dateStr
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.

2 participants