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

[Issue #1619] Update Google Analytics to add App Router support #1879

Merged
merged 2 commits into from
Apr 30, 2024

Conversation

btabaska
Copy link
Collaborator

Summary

Fixes #1619

Time to review: 4 mins

Changes proposed

Added GA to App Router

Context for reviewers

Next.js requires a different GA setup for pages using the App Router compared to the page router.
https://nextjs.org/docs/messages/next-script-for-ga

Additional information

Screenshots, GIF demos, code examples or output to help show the changes working as expected.

Copy link

Coverage report for ./frontend

St.
Category Percentage Covered / Total
🟢 Statements 83.89% 854/1018
🟡 Branches 64.9% 220/339
🟡 Functions 75.34% 165/219
🟢 Lines 83.89% 797/950

Test suite run success

166 tests passing in 55 suites.

Report generated by 🧪jest coverage report action from e219dc3

@btabaska btabaska changed the title [Issue #1619] Add transformations for assistance listing table [Issue #1619] Update Google Analytics to add App Router support Apr 30, 2024
@btabaska btabaska merged commit fbf5100 into main Apr 30, 2024
10 checks passed
@btabaska btabaska deleted the btabaska/1619-add-google-analytics-to-app-router branch April 30, 2024 19:59
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.

[Task]: Spike for Search page tracking set up for GA
2 participants