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

GoogleAnalytics 기본 #67

Merged
merged 4 commits into from
Aug 29, 2021
Merged

Conversation

josworks27
Copy link
Member

@josworks27 josworks27 commented Aug 22, 2021

Description

구글 애널리틱스 아카데미 자료를 바탕으로 기본적인 내용을 요약 정리하였습니다.

1단원의 주요 내용을 정리하였으며, 1.4의 경우 따로 요약할 내용이 없어 원문 링크를 추가하였습니다.

Help Wanted 👀

Related Issues

resolve #37
fix #37

Checklist ✋

  • 모든 변경점들을 확인했으며 적절히 설명했습니다.
  • 빌드가 정상적으로 수행됨을 확인했습니다. (yarn build)

* 구글 애널리틱스 기본 생성
* relates to EveryAnalytics#37
* 폴더 변경 / blog => handbook
* relates to EveryAnalytics#37
Copy link
Contributor

@hiphapis hiphapis left a comment

Choose a reason for hiding this comment

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

굿굿!

@josworks27
Copy link
Member Author

@all-contributors please add @josworks27 for code

@allcontributors
Copy link
Contributor

@josworks27

I've put up a pull request to add @josworks27! 🎉

Copy link
Member

@greatSumini greatSumini left a comment

Choose a reason for hiding this comment

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

디렉토리명을 amplitude와 동일하게 ga-basics로 맞추는건 어떨까요?? google-analytics-basics도 좋구요 ㅎㅎ

@josworks27
Copy link
Member Author

@greatSumini 네 좋아요 ~ 핸드북 목차에 표시되는 만큼 풀 네임으로 google-analytics-basics 가 좋지 않을까 싶은데 어떠세요 ?

* 디렉토리 변경
* 내용 추가(구글 애널리틱스 설정, 계정 보기 필터를 설정하는 방법)
* relates to EveryAnalytics#37
@jiyoon1156
Copy link
Member

google-analytics-basics 좋습니다!!

@alledy
Copy link
Member

alledy commented Aug 28, 2021

요거 저만 그런건진 모르겠는데 이 pr checkout 해서 빌드하면 Expected corresponding JSX closing tag for <head> 요런 에러가 나오는데 한 번 확인해주실 수 있을까요? 추가된 md 파일 삭제하고 빌드하면 정상적으로 빌드됨니당🤔

* MD 내 Tag로 인식되는  <head> 수정
@josworks27
Copy link
Member Author

josworks27 commented Aug 28, 2021

@alledy

내용 중에 <head> 라는 단어가 태그로 인식되는 이슈가 있어서 수정하였습니다. mdx가 아닌 md에서 꺽쇠가 태그로 인식된다는 것을 확인하지 못하였습니다. 감사합니다.

@josworks27
Copy link
Member Author

일반 마크다운 문서에서 태그 형식의 문장이 태그로 인식되는 것과 관련하여
docusaurus에서 MDX 파싱 엔진을 사용하고 있는 것으로 확인하였습니다.

The standard Markdown syntax is supported, and we use MDX as the parsing engine, which can do much more than just parsing Markdown, like rendering React components inside your documents.
https://docusaurus.io/docs/markdown-features

mdx를 지원하는 만큼 마크다운 내에서 jsx 코드를 적극적으로 사용해보면 좋을 것 같습니다.

@alledy
Copy link
Member

alledy commented Aug 28, 2021

오호 확장자가 mdx가 아니어도 그냥 꺾쇠만 들어가면 그렇게 인식하는 모양이군요👀 빠른 수정 감사합니다👍

@greatSumini
Copy link
Member

네 좋아요 ~ 핸드북 목차에 표시되는 만큼 풀 네임으로 google-analytics-basics 가 좋지 않을까 싶은데 어떠세요 ?

@josworks27 너무 좋습니다!!

@josworks27 josworks27 merged commit 898112a into EveryAnalytics:main Aug 29, 2021
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.

Google Analytics 기본
5 participants