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

구글 태그 인식을 위한 script 코드 추가, 사이드바에서 Toast가 잘려 보이는 이슈 해결 #593

Merged
merged 4 commits into from
Sep 13, 2023

Conversation

inyeong-kang
Copy link
Member

@inyeong-kang inyeong-kang commented Sep 12, 2023

🔥 연관 이슈

close: #592
close: #475

📝 작업 요약

구글 태그 인식을 위한 script 코드 추가

⏰ 소요 시간

10분

🔎 작업 상세 설명

기존 코드로 구글 태그가 인식되지 않아 script를 추가함.
React 같은 SPA 앱인 경우에는 일반 html 문서와 렌더링 방식이 다르기 때문에, 컴포넌트를 별도로 만들어서 App.tsx 에서 import 해야 제대로 구글 태그가 인식된다고 하네요..!
이를 위해 react-gtm-module이라는 모듈 설치해줬습니다.

🌟 논의 사항

참고자료는 아래와 같습니다.
https://neep305.tistory.com/61
https://juunone.netlify.app/web/gtm/
https://goo-gy.github.io/2021-04-28-google-tag-manager

@github-actions
Copy link

github-actions bot commented Sep 12, 2023

⚡️ Lighthouse report!

Category Score
🟠 Performance 51
🟢 Accessibilty 92
🟢 SEO 92
🟠 PWA 89
Category Score
🟢 First Contentful Paint 0.7 s
🔴 Largest Contentful Paint 4.6 s
🔴 Total Blocking Time 4,290 ms
🟢 Cumulative Layout Shift 0
🟠 Speed Index 4.6 s

Copy link
Collaborator

@Gilpop8663 Gilpop8663 left a comment

Choose a reason for hiding this comment

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

확인했습니당~

fe-리뷰완

</PostOptionProvider>
</ThemeProvider>
</QueryClientProvider>
</ErrorBoundaryForTopClass>
Copy link
Member Author

@inyeong-kang inyeong-kang Sep 13, 2023

Choose a reason for hiding this comment

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

에러가 나는 경우에도 구글 태그가 있어야 지속적인 사용자 추적이 가능해서 최상단에 구글 태그를 두었습니다.

@inyeong-kang inyeong-kang changed the title 구글 태그 인식을 위한 script 코드 추가 구글 태그 인식을 위한 script 코드 추가, 사이드바에서 Toast가 잘려 보이는 이슈 해결 Sep 13, 2023
Copy link
Collaborator

@chsua chsua left a comment

Choose a reason for hiding this comment

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

좋아요~~
고생하셨습니다!

@chsua chsua merged commit 3afed95 into dev Sep 13, 2023
1 check passed
@woo-chang woo-chang deleted the fix/#592 branch September 13, 2023 07:51
@woo-chang woo-chang restored the fix/#592 branch September 13, 2023 07:52
@woo-chang woo-chang deleted the fix/#592 branch September 13, 2023 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants