Skip to content

Conversation

@gudusol
Copy link
Contributor

@gudusol gudusol commented Apr 2, 2025

📌 Related Issue Number


Checklist

  • 🎋 base 브랜치를 제대로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏗️ 빌드는 성공했나요? (pnpm build)
  • 🧹 불필요한 코드는 제거했나요? e.g. console.log
  • 🙇‍♂️ 리뷰어를 지정했나요?
  • 🏷️ 라벨은 등록했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 3월 31일 QA 회의 진행중 나온 내용들 반영
    • 오타 및 UI 버그 수정
    • 한 걸음 더 <-> 포인팅 순서 변경
  2. GA 코드 수정

💡 New Insights & Learnings

  • GA 이벤트에 params를 넣어서 수집하려면, params에 대한 맞춤 설정이 필요하다고 함.
  • 이걸 의도한 것은 아녔으므로 params 일부 삭제했고, 공통 pathname params도 삭제함. 더이상 usePathname 안써도 되니 utils로 옮김
  • 대신, params로 넘기던 정보 몇개를 name자체에 넣어서 데이터를 수집함

@gudusol gudusol added 🐞 Fix 버그 수정 🐽 태승 labels Apr 2, 2025
@gudusol gudusol self-assigned this Apr 2, 2025
@gudusol gudusol merged commit 80c803d into main Apr 2, 2025
1 check passed
@gudusol gudusol deleted the fix/service/qa-#64 branch April 2, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[fix] QA 3차 반영

2 participants