-
Notifications
You must be signed in to change notification settings - Fork 8
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
Find a better way to include Azure analytics #472
Comments
Relevant issue on the application insights repo: microsoft/ApplicationInsights-JS#1076 |
Related to #711 |
We need to discuss what we need here |
Had to remove the previous work due to pnpm v7 peer deps issues. Adding a link to the commit here for future reference. #bekindtofutureself We need to investigate this furter, and it might not even make sense to use Azure analytics |
Do we still need/want Azure Insights? Especially now that we are working on enabling Dynatrace |
#1309 Decide on what to do monitoring-wise |
This commit removed the MS application insight because it had such a high impact on first JS shared by all
See the difference using Next.js's CLI
Look into a better way to include this.
The text was updated successfully, but these errors were encountered: