Skip to content

feat: Vercel Web Analytics 적용#34

Merged
bye-on merged 4 commits intomainfrom
feat/add-vercel-analytics
May 29, 2025
Merged

feat: Vercel Web Analytics 적용#34
bye-on merged 4 commits intomainfrom
feat/add-vercel-analytics

Conversation

@nijuy
Copy link
Owner

@nijuy nijuy commented May 26, 2025

#️⃣ 관련 이슈

💻 주요 변경 사항

💬 To. 리뷰어

테스트를 위해 접속했을 때 수집되는 데이터를 최대한 줄이고 싶어서 이 PR은 가장 마지막에 머지할 예정입니다
참고 부탁드립니다..! 🙇‍♀️

잠금 파일이 포맷팅되면서 변경사항이 2000줄 넘게 잡혔네요 🤔
프리티어 설정 중 printWidth: 80 으로 인해서 줄바꿈이 추가로 생긴 거 같아요.
잠금 파일에 포맷팅 적용 안 되게 ignore 파일 추가해뒀어야 하는데 죄송합니다-!

// 원래 이렇게 작성되어 있던게
  html2canvas@1.4.1:
    resolution: {integrity: ...}
    engines: {node: '>=8.0.0'}

// 이렇게 바뀌면서 변경사항이 늘어났어요
  html2canvas@1.4.1:
    resolution:
      {
        integrity: ...,
      }
    engines: { node: '>=8.0.0' }

@nijuy nijuy self-assigned this May 26, 2025
@nijuy nijuy linked an issue May 26, 2025 that may be closed by this pull request
1 task
@bye-on
Copy link
Collaborator

bye-on commented May 26, 2025

image

Vercel Analytics를 사용하려면 대시보드에서 Enable을 눌러야하는 것 같은데, 해당 PR이 머지되면 Enable 할까요??

@nijuy
Copy link
Owner Author

nijuy commented May 26, 2025

@bye-on 아 맞네용 까먹을 뻔 했는데 감사합니다 굿굿

image

문서 보니 대시보드에서 설정 켜는게 첫 스텝이라 머지 전에 활성화부터 해야겠네요
기능 개발 끝나면 채원님이 직접 활성화~ 머지하시는 게 편하세요 아님 설정하신 거 알려주시면 제가 확인하고 합칠까요?

@bye-on
Copy link
Collaborator

bye-on commented May 27, 2025

제가 활성화한 뒤에 머지할게요!!

@bye-on bye-on merged commit 3dd6ebf into main May 29, 2025
1 check passed
@bye-on bye-on deleted the feat/add-vercel-analytics branch May 29, 2025 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Vercel Web Analytics 도입

3 participants