Skip to content

밴드 홈 추천 질문 제거 - #273

Merged
njm1250 merged 1 commit into
devfrom
fix/remove-main-assistant-presets
Oct 24, 2026
Merged

njm1250 merged 1 commit into
devfrom
fix/remove-main-assistant-presets

Conversation

@njm1250

@njm1250 njm1250 commented Oct 24, 2026

Copy link
Copy Markdown
Member

⏱ 소요 시간

  • 리뷰 예상 시간: 5분

📌 작업 요약

밴드 홈에서는 자유 질문 입력만 제공하고, 추천 질문은 AI 질문 화면에서만 노출하도록 정리


📝 작업 내용

  1. 밴드 홈 AssistantPanel의 추천 질문 조회 및 추천 칩 제거
  2. AI 질문 화면의 추천 질문 기능은 기존대로 유지
  3. 홈과 AI 질문 화면의 역할에 맞게 AssistantPanel 테스트 수정

🚨 주요 고민 및 해결 과정

문제

동일한 추천 질문이 밴드 홈과 AI 질문 화면에 중복 노출되어 메인 화면이 복잡해짐

해결 과정

메인 화면은 자유 질문을 빠르게 시작하는 입력창만 유지하고, 추천 질문 탐색은 AI 질문 화면으로 모음. 추천 질문 API와 대화 로직은 건드리지 않음


📑 참고 문서/ ADR

  • 없음

💬 리뷰 요구사항

  • 밴드 홈에 자유 질문 입력만 남기는 구성이 적절한지 확인 요청
  • 추천 질문이 AI 질문 화면에서 기존대로 노출되고 동작하는지 확인 요청

✅ 검증

  • AssistantPanel 테스트 25개 통과
  • 변경 파일 ESLint·Prettier 통과
  • 전체 TypeScript 검증은 최신 dev의 frontend/src/features/place-create/ui/PlaceCreateModal.tsx:47 JSX 문법 오류로 실패함. 이번 PR 변경 파일과는 무관한 기준 브랜치 오류

@vercel

vercel Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
band-co Error Error Oct 24, 2026 3:24pm UTC

@coderabbitai

coderabbitai Bot commented Oct 24, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@njm1250
njm1250 merged commit 110f2e6 into dev Oct 24, 2026
3 of 4 checks passed
@njm1250
njm1250 deleted the fix/remove-main-assistant-presets branch October 24, 2026 15:25

This branch had an error being deployed

1 failed deployment
Preview — ae69d749 Deployed Oct 24, 2026 by vercel[bot]
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.

1 participant