Skip to content

feat(home): 이벤트 종료에 맞춰 피드백 배너와 폼 링크 교체 - #259

Merged
minbros merged 1 commit into
devfrom
feature/feedback-banner-after-event
Nov 8, 2026
Merged

minbros merged 1 commit into
devfrom
feature/feedback-banner-after-event

Conversation

@parkjuyeong0312

Copy link
Copy Markdown
Member

변경 내용

  • 홈 배너에서 싸이버거 이벤트 문구·기간·EVENT 배지·버거 사진을 빼고, 참여 유도 문구와 리뷰 3D 아이콘으로 바꿨습니다.
    • 소제목: 우때 써보니 어떠셨나요? 짧은 한마디도 큰 힘이 돼요!
    • 제목: 우때는 여러분의 피드백으로 / 더 좋은 여행계획 도구로 발전 중!
    • 버튼: 지금 피드백 남기기 >
  • 피드백 폼 링크(FEEDBACK_FORM_URL)를 이벤트 문항을 뺀 새 폼으로 바꿨습니다. 홈 배너·사이드바·모바일 헤더 메뉴가 같은 링크를 씁니다.
  • 쓰지 않는 이벤트 배너 이미지 3개(burger.webp·desktop.png·mobile.png)를 지우고 리뷰 아이콘 WebP(360px, 무손실)를 추가했습니다.
  • 배너 색은 디자인 토큰(--color-primary-subtle·--color-text-default·--color-primary-default)으로 바꿨습니다.
  • 시안: 마케팅 / 방목록 배너광고 / 피드백 배너 광고

변경 이유

  • 피드백 이벤트(~10/25)가 끝났는데 홈 배너와 피드백 폼에 이벤트 안내가 남아 있었습니다.

테스트

  • npm run lint — 변경 파일 오류 없음. 저장소 폴더 안의 다른 로컬 worktree(.claude/worktrees/…/tailwind.config.js) 오류 1건은 이 PR과 무관합니다.
  • npm test — 홈 테스트 23개 통과. 전체 실패 58건(ExpenseEditor·ExpensePanel·MobilePlaceSearchScreen)은 이 변경을 빼고 dev에서 돌려도 똑같이 실패합니다.
  • npm run build (+ npx tsc --noEmit 통과)
  • 화면 확인: 로컬 임시 미리보기에서 데스크톱 1136px·모바일(is-mobile-device) 353px 배너가 시안과 같고, 링크가 새 폼으로 열립니다.
  • 그 외 수동 검증: 새 폼 응답자 화면에서 헤더·문항과 이벤트 문구 제거를 확인했습니다.

🤖 Generated with Claude Code

- 싸이버거 이벤트 문구·기간·배지·버거 사진을 빼고 참여 유도 문구와 리뷰 3D 아이콘으로 교체
- 피드백 폼 링크를 이벤트 문항을 뺀 새 폼으로 변경(배너·사이드바·헤더 공통)
- 쓰지 않는 이벤트 배너 이미지(burger·desktop·mobile) 삭제

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
frontend-server Ready Ready Preview Nov 4, 2026 5:04pm UTC

Request Review

@sonarqubecloud

sonarqubecloud Bot commented Nov 4, 2026

Copy link
Copy Markdown

@minbros
minbros merged commit ff1b4fe into dev Nov 8, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 910e2cdf Deployed Nov 4, 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.

2 participants