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

[NDD-150] 구글 애널리틱스 설정 완료 (0.5h/1h) #48

Merged
merged 3 commits into from
Nov 15, 2023

Conversation

milk717
Copy link
Collaborator

@milk717 milk717 commented Nov 15, 2023

NDD-150 Powered by Pull Request Badge

Why

우선 구글 애널리틱스 태그 연결만 진행했습니다.
각 페이지별 추적이나 행동 추적 이벤트는 좀 더 개발이 진행된 후에 다는 것이 좋다고 판단했습니다.

How

  • GTAG는 웹사이트에서 개발자모드를 켜서 확인할 수 있는 코드라서 env에 숨기지 않아도 되지만... 뭔가가 뭔가라는 팀원들의 의견을 반영해서 env파일에 넣었습니다.
  • env 파일을 설정하기 위해 webpack에 dotenv 플러그인을 추가했습니다.
  • GA 추적 태그를 설정하는 코드는 일단 index.ts에 넣어놨는데 다른곳에 넣는 것이 좋다는 의견이 있으면 이동하도록 하겠습니다~

Result

image

@milk717 milk717 self-assigned this Nov 15, 2023
@milk717 milk717 added FE 프론트엔드 코드 변경사항 feature 새로운 기능이 추가 된 경우 chore 의존성이 변경된 경우 labels Nov 15, 2023
Copy link
Collaborator

@Yoon-Hae-Min Yoon-Hae-Min left a comment

Choose a reason for hiding this comment

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

10줄짜리 PR 이건 정멀 귀하네요 저희 env 관리에 env파일 내용도 한번 올려주세요!!

Copy link
Collaborator

@adultlee adultlee left a comment

Choose a reason for hiding this comment

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

크으 고생하셨습니다!!

@adultlee
Copy link
Collaborator

아무리봐도 0.5 짜리는 아니었겠지만... 정말 고생하셨어요!!!

@milk717
Copy link
Collaborator Author

milk717 commented Nov 15, 2023

아무리봐도 0.5 짜리는 아니었겠지만... 정말 고생하셨어요!!!

0.1은 너무한 것 같아서 살짝 부풀렸습니다 ㅎㅅㅎ

@milk717 milk717 merged commit 0518004 into dev Nov 15, 2023
1 check passed
@delete-merged-branch delete-merged-branch bot deleted the feature/NDD-150 branch November 15, 2023 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore 의존성이 변경된 경우 FE 프론트엔드 코드 변경사항 feature 새로운 기능이 추가 된 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants