-
Notifications
You must be signed in to change notification settings - Fork 1
[FIX] 메인페이지의 토론 템플릿 데이터 수정 #366
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Walkthrough메인 페이지 토론 템플릿 상수 데이터가 갱신되었습니다. ONE과 TWO 항목의 예선/본선/결승에 대한 인코딩된 공유 URL 페이로드가 교체되었고, TWO의 제목이 '노곧떼'에서 '노곳떼'로 수정되었습니다. 코드 시그니처 변경은 없습니다. Changes
Sequence Diagram(s)Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Assessment against linked issues
Suggested labels
Suggested reviewers
Poem
✨ Finishing Touches
🧪 Generate unit tests
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. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/constants/debate_template.ts (1)
15-15: 환경변수 VITE_SHARE_BASE_URL 정의 및 기본값 추가 필요
루트에.env*파일이 없으며,import.meta.env.VITE_SHARE_BASE_URL가 설정되지 않을 경우 공유 링크가 깨집니다..env,.env.production또는 CI 환경 설정 등에 해당 변수를 정의하거나 코드 내에서 기본값(fallback)을 추가하세요.
🧹 Nitpick comments (4)
src/constants/debate_template.ts (4)
14-16: 동일 명칭의 유틸 함수와 시그니처 불일치로 인한 혼동 가능성이 파일의
createTableShareUrl(encodeData: string)은 인코딩된 문자열을 그대로 붙입니다. 반면src/util/arrayEncoding.ts의 동명 함수는DebateTableData를 받아 내부에서 인코딩합니다. 동명이지만 의미가 달라 혼동 여지가 큽니다. 최소한 네이밍을 구분(예:createShareUrlFromEncoded)하거나 JSDoc으로 역할을 명시해 주세요.
30-31: 라벨 표기 통일(Nit): ‘3vs3’ ↔ ‘3:3’ 혼용현재 첫 섹션은 ‘3vs3’, 다른 섹션은 ‘3:3’을 사용합니다. 표기 통일을 권장합니다(예: ‘3:3’).
적용 예시:
- label: '3vs3 통상토론대회 형식', + label: '3:3 통상토론대회 형식',Also applies to: 256-266
122-126: 라벨 띄어쓰기 통일(Nit): ‘자유토론’ ↔ ‘자유 토론’ 혼용‘2:2 자유토론’(라인 122 인근)과 ‘2:2 자유 토론’(라인 147 인근)이 혼재합니다. 한 가지로 통일해 주세요(예: ‘자유토론’).
예시:
- label: '2:2 자유 토론', + label: '2:2 자유토론',Also applies to: 147-151
22-23:DEBATE_TEMPLATE에as const satisfies DebateTemplateList적용 제안현재
package.json상의 TypeScript 버전이 ^5.7.2로(TS 4.9+)satisfies사용이 가능합니다. 다음과 같이 변경해 불변성과 타입 적합성을 강화하세요:-export const DEBATE_TEMPLATE: DebateTemplateList = { +export const DEBATE_TEMPLATE = { … -}; +} as const satisfies DebateTemplateList;270번 줄에도 동일하게 적용 가능합니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/constants/debate_template.ts(2 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
src/constants/debate_template.ts (1)
src/util/arrayEncoding.ts (1)
createTableShareUrl(23-29)
🔇 Additional comments (3)
src/constants/debate_template.ts (3)
220-220: 오탈자 수정 완료: 잔여 ‘노곧떼’ 없음 확인
전역 검색(rg -n "노곧떼|노곳떼") 결과, 잘못된 ‘노곧떼’ 표현은 더 이상 존재하지 않습니다. 머지 승인합니다.
86-87: 검토 완료 — 변경된 ‘열린 토론대회 예선, 본선’ 페이로드와 라벨/맥락(중앙선거방송토론위원회)이 요구사항에 일치하며, 스테이징에서 디코딩된 테이블 구성도 확인되었습니다.
92-93: LGTM — ‘열린 토론대회 결승’ 페이로드 업데이트 적정합니다.
공유 링크를 디코딩해 라운드/시간 배치가 최신 ‘대한민국 열린 토론대회’ 규격과 일치하는지 최종 시각 확인 권장.
🚩 연관 이슈
closed #365
📝 작업 내용
노곧떼 -> 노곳떼 로 수정
토론 템플릿 인코딩 데이터 변경
🏞️ 스크린샷 (선택)
🗣️ 리뷰 요구사항 (선택)
Summary by CodeRabbit
Bug Fixes
Chores