Conversation
Merge pull request #432 from debate-timer/main
* feat: 타이머 오버뷰에서 토론자 순번 번역 추가 * feat: 헤더의 테이블정보와 타이틀 번역 추가 * refactor: 토론 정보 수정시 언어에 따른 기본 value 번역되도록 수정 * fix: 직접입력의 키가 불일치하여 번역이 적용되지 않는 문제 해결 * refactor: 찬성, 반대, 중립을 선택하는 드롭박스가 번역되도록 사용처에 번역 작업 추가 * refactor: 헤더 컴포넌트 내부에서 번역 작업하던 구조를 사용처에서 하도록 개선 * refactor: 팀 이름 키가 있는 경우 번역되도록 번역 작업 추가 * refactor: 기본 입력으로 생성된 각각의 타이머 경우에도 팀명이 번역되도록 번역 작업 추가 * refactor: Schedule -> Timetable로 수정 * refactor: 팀 이름 검증 규칙 번역키, 밸류 수정 * refactor: 팀 이름 15자 이내로 변경 * refactor: 번역 오류 수정 및 대소문자 수정 * refactor: 분과 초를 함께 쓰는 경우 기호 버전으로 min -> m 사용하도록 수정 * refactor: 이전차례, 다음차례 번역 수정 * refactor: 기본 템플릿에 설정된 발언 유형 번역 추가 * refactor: 1회당 발언 시간, 팀당 발언 시간 번역 수정 * refactor: 좌측 영역의 정보의 길이 때문에 레이아웃이 바뀌지 않도록 수정 * refactor: 새로운 템플릿 신청 관련 번역 수정 * refactor: 동사형으로 들어가야 하는 번역 수정 * refactor: 사용하지 않는 prop 삭제 * refactor: 번역 수정
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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.
Code Review
This pull request updates localization files for English and Korean, increases the maximum team name length to 15 characters, and refactors components to centralize translation logic. Feedback focuses on several instances where user-inputted strings (such as speaker names and custom speech types) or already-translated variables are incorrectly wrapped in the translation function t(), which may cause unexpected display issues or redundant processing.
| <ClearableInput | ||
| id="speech-type-time-based" | ||
| value={speechTypeTextValue} | ||
| value={t(speechTypeTextValue)} |
| {speaker && ( | ||
| <span className="font-medium"> | ||
| {t(' | {{speaker}} 토론자', { speaker })} | ||
| {t(' | {{speaker}} 토론자', { speaker: t(speaker) })} |
| <ClearableInput | ||
| id="speaker" | ||
| value={speaker} | ||
| value={t(speaker)} |
| <p className="truncate text-[20px] xl:text-[28px]"> | ||
| {teamName && t('{{team}} 팀', { team: teamName })} | ||
| <p className="min-w-0 flex-1 truncate text-[20px] xl:text-[28px]"> | ||
| {teamName && t('{{team}} 팀', { team: t(teamName) })} |
| {teamName && t('{{team}} 팀', { team: t(teamName) })} | ||
| {item.speaker && | ||
| t(' | {{speaker}} 토론자', { speaker: item.speaker })} | ||
| t(' | {{speaker}} 토론자', { speaker: t(item.speaker) })} |
| <section className="flex flex-row items-center justify-center space-x-[24px]"> | ||
| <h1 className="text-[48px] font-bold text-default-black xl:text-[64px]"> | ||
| {teamName} | ||
| {t(teamName)} |
🚩 연관 이슈
없음
📝 작업 내용
자동화 범위 이외의 번역 추가 및 검증 에러 해결 기능 main merge 진행
🏞️ 스크린샷 (선택)
🗣️ 리뷰 요구사항 (선택)