-
Notifications
You must be signed in to change notification settings - Fork 48
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
check for process.env.VERCEL_ANALYTICS_ID #630
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
What issue is this PR resolving? |
the issue where it shows a bunch of red text in my console when i npm run dev |
The analytics ID is currently injected into the build at build time in Line 9 in 6f43b29
SvelteKit generates types for the available environment variables at build time. I think it might be possible to extend the functionality described here: https://kit.svelte.dev/docs/modules#$env-dynamic-public. |
- Refactored analytics library: factored out `send.ts` into `vitals.ts`. Addresses <#630 (comment)>. Thanks @jaasonw for kicking off this PR :)
- Refactored analytics library: factored out `send.ts` into `vitals.ts`. - Added env variables to pass `website_check.yaml` workflow. Addresses <#630 (comment)>. Thanks @jaasonw for kicking off this PR :)
f952e09
to
ee8622e
Compare
- Refactored analytics library: factored out `send.ts` into `vitals.ts`. - Added env variables to pass `website_check.yaml` workflow. Addresses <#630 (comment)>. Thanks @jaasonw for kicking off this PR :)
7b9ee10
to
8327b18
Compare
8327b18
to
ece52d5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After some light reorganization, this PR lgtm!
oh |
resolves 400 bad request error when local dev environment