[FIX] 열린토론대회 토론 템플릿 데이터 변경

[jaeml06]

🚩 연관 이슈

closed #372

📝 작업 내용

열린토론대회 토론 템플릿 데이터 변경

🏞️ 스크린샷 (선택)

🗣️ 리뷰 요구사항 (선택)

Summary by CodeRabbit

• 버그 수정
  • TWO 섹션의 ‘열린 토론대회 결승’ 액션 링크 데이터를 갱신하여 올바른 대상에 연결되도록 개선했습니다.

[coderabbitai]

Walkthrough

src/constants/debate_template.ts의 DEBATE_TEMPLATE 중 TWO 섹션 내 "열린 토론대회 결승" 액션의 href 데이터 페이로드가 새로운 인코딩 문자열로 교체되었습니다. 그 외 구조, 타입, 함수, 제어 흐름 변경은 없습니다.

Changes

Cohort / File(s)	Change Summary
토론 템플릿 데이터 갱신 src/constants/debate_template.ts	DEBATE_TEMPLATE의 TWO 섹션 "열린 토론대회 결승" 항목의 href 인코딩 데이터 문자열 교체. 기타 로직 및 구조 변화 없음.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• [FIX] 메인페이지의 토론 템플릿 데이터 수정 #366: 같은 파일의 TWO 섹션 결승 항목 href 페이로드를 수정하며, 추가로 다른 페이로드와 명칭 변경도 포함된 관련 변경.

Suggested labels

fix

Suggested reviewers

• useon

Pre-merge checks (4 passed, 1 warning)

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues Check	⚠️ Warning	링크된 이슈 #372의 주요 목표 중 첫 번째인 토론 템플릿 데이터 수정은 올바르게 반영되었으나 두 번째 목표인 메인 페이지 GIF 수정이 코드 변경에 포함되지 않았습니다. 따라서 이슈에서 요구한 모든 작업이 완성되지 않았습니다.	메인 페이지 GIF 수정 작업이 누락된 상태이므로 해당 리소스 업데이트를 추가해 이슈 #372의 모든 요구사항을 충족하도록 보완해야 합니다.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	PR 제목 “[FIX] 열린토론대회 토론 템플릿 데이터 변경”은 변경된 핵심 파일인 debate_template.ts에서 템플릿 데이터만 수정된 사실을 정확히 반영하고 있어 주요 변경점을 명확하게 전달합니다. 제목이 간결하면서도 코드 변경 범위를 잘 서술하고 있어 동료가 히스토리를 살필 때도 혼동 없이 파악할 수 있습니다.
Out of Scope Changes Check	✅ Passed	변경된 파일이 debate_template.ts 하나에 한정되어 있으며 이슈 범위를 벗어난 다른 기능 추가나 수정이 전혀 없어 불필요한 코드가 포함되어 있지 않습니다.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

Poem

꼬박꼬박 토론 길, 링크만 살짝 손봤지요
토끼는 귀를 쫑긋, 결승으로 바로 뛰어가요 🐇
바이트 한 줄 갈아끼우고
길 잃지 않게 길잡이 표지 붙였지요
이제 클릭! 폴짝—결승장으로 고!

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

✨ Finishing Touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/#372

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

src/constants/debate_template.ts (1)

14-16: 공유 URL 생성 함수 — 슬래시 정규화 및 파라미터 명확화(옵션)

BASE_URL에 슬래시가 포함된 환경도 안전하게 처리하고, 인자가 “이미 인코딩된 데이터”임을 이름으로 명확히 하길 제안합니다.

다음과 같이 소폭 정리 가능:

-function createTableShareUrl(encodeData: string): string {
-  return `${import.meta.env.VITE_SHARE_BASE_URL}/share?data=${encodeData}`;
-}
+function createTableShareUrl(encodedData: string): string {
+  const base = (import.meta.env.VITE_SHARE_BASE_URL || '').replace(/\/+$/, '');
+  return `${base}/share?data=${encodedData}`;
+}

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 183e24f and d3200b3.

📒 Files selected for processing (1)

• src/constants/debate_template.ts (1 hunks)

🔇 Additional comments (2)

src/constants/debate_template.ts (2)

22-22: GIF 자산 변경 없음 확인
public/spinner.gif만 존재하며 이 PR에서 해당 파일이 수정되지 않았습니다. 이슈 #372의 메인 페이지 GIF 수정은 별도 PR로 처리된 것으로 보입니다.

---

90-93: href 페이로드 검증 완료 새 payload가 정상적으로 디코딩되었고 예상 스키마(id, info, table)와 일치합니다.