Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FE] feat: GA 이벤트 추가 #721

Merged
merged 11 commits into from
Oct 6, 2023
Merged

[FE] feat: GA 이벤트 추가 #721

merged 11 commits into from
Oct 6, 2023

Conversation

Leejin-Yang
Copy link
Collaborator

Issue

✨ 구현한 기능

  • GA 이벤트 추가
    • 상품 정렬, 리뷰 정렬, 리뷰 작성 버튼, 리뷰 등록
    • 꿀조합 정렬, 작성 버튼, 등록
    • 목록 페이지 카테고리 버튼
    • 검색 페이지 검색

📢 논의하고 싶은 내용

  • 배너, 랭킹 등 링크로 되어있는 친구들은 이벤트 추가하지 않았습니다. 이벤트 등록을 하려면 button, navigate로 해야할 거 같아요. 이 부분에 대해서 의견주세요.!
  • 더 추가해야할 이벤트 있다면 남겨주세요 😊

🎸 기타

  • 특이 사항이 있으면 작성합니다.

⏰ 일정

  • 추정 시간 : 2시간
  • 걸린 시간 : 1시간

@github-actions
Copy link

github-actions bot commented Oct 3, 2023

Test Results

8 tests  ±0   8 ✔️ ±0   6s ⏱️ ±0s
3 suites ±0   0 💤 ±0 
1 files   ±0   0 ±0 

Results for commit 4c99360. ± Comparison against base commit 70c9c4b.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굿굿 좋습니다
근데 리뷰 작성이나 꿀조합 등록 버튼은 우리가 작성되는 것을 눈으로 확인할 수 있기 때문에 GA를 달 필요가 없을 것 같아요
근데 Link 태그에 onClick을 달 수는 없나요?? 랭킹에 ga 달면 좋을 것 같은데 다 링크라서..

@Leejin-Yang
Copy link
Collaborator Author

@xodms0309
이벤트 메서드를 실행해야해서 이벤트 핸들러로 navigate 함수와 같이 쓰더라구요.. 검색은 pageview로 볼 수 있지만ㅎㅎ..
버튼으로 바꾼다 또는 링크로 냅둔다 중에 선택해야 할거 같네요

Copy link
Collaborator

@hae-on hae-on left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨어요!
궁금한게 있는데 Link에 onClick 있으면 보이스오버가 어떻게 읽나요???
실험해보시죠

@Leejin-Yang
Copy link
Collaborator Author

@hae-on
보이스오버를 실행합니다 버튼 버튼

Copy link
Collaborator

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

빨리 production 고고

@Leejin-Yang Leejin-Yang merged commit 86e9430 into develop Oct 6, 2023
3 checks passed
@Leejin-Yang Leejin-Yang deleted the feat/issue-717 branch October 6, 2023 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] feat: 애널리틱스 이벤트 추가
3 participants