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

565-feat: Connect google analytics #676

Closed
wants to merge 5 commits into from

Conversation

ansivgit
Copy link
Collaborator

@ansivgit ansivgit commented Dec 4, 2024

What type of PR is this? (select all that apply)

  • πŸ• Feature
  • πŸ› Bug Fix
  • 🚧 Breaking Change
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ“ Documentation Update

Description

Added custom Analitics.tsx component with GA tag.

Related Tickets & Documents

Added/updated tests?

  • πŸ‘Œ Yes
  • πŸ™…β€β™‚οΈ No, because they aren't needed
  • πŸ™‹β€β™‚οΈ No, because I need help

[optional] Are there any post deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

@ansivgit ansivgit self-assigned this Dec 4, 2024
@github-actions github-actions bot changed the title Feat/565 connect google analytics 565-feat: Connect google analytics Dec 4, 2024
export const Analytics = () => {
return (
<>
<Script strategy="afterInteractive" src="https://www.googletagmanager.com/gtag/js? id=G-THXM004SPR" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

extra space ? id

@@ -23,6 +23,7 @@
"node": ">=20.x"
},
"dependencies": {
"@next/third-parties": "^15.0.2",
Copy link
Collaborator

Choose a reason for hiding this comment

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

we don't need this package since you're not using GoogleTagManager, right?

return (
<>
<Script strategy="afterInteractive" src="https://www.googletagmanager.com/gtag/js? id=G-THXM004SPR" />
<Script strategy="afterInteractive">
Copy link
Collaborator

Choose a reason for hiding this comment

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

we can omit the strategy prop, since it defaults to afterInteractive

Copy link

Lighthouse Report:

  • Performance: 73 / 100
  • Accessibility: 99 / 100
  • Best Practices: 100 / 100
  • SEO: 100 / 100

View detailed report

Copy link

Lighthouse Report:

  • Performance: 73 / 100
  • Accessibility: 99 / 100
  • Best Practices: 100 / 100
  • SEO: 100 / 100

View detailed report

@ansivgit ansivgit closed this Dec 23, 2024
@ansivgit ansivgit deleted the feat/565-connect-google-analytics branch December 23, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Connect Google Analytics
3 participants