search

vercel / speed-insights

Vercel Speed Insights package
https://vercel.com/docs/speed-insights
Apache License 2.0
62 stars 9 forks source link

Make access defensive and fix environment variable reference. #87

Closed emspishak closed 3 days ago

emspishak commented 1 week ago

process may not exist in all frameworks, this ensures things will continue to work as expected.

process.env[someVar] works in dev mode but not production, so this switches to process.env.NEXT_PUBLIC_....

📓 What's in there?

🧪 How to test?

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
speed-insights-astro ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 6:08pm
speed-insights-nextjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 6:08pm
speed-insights-nuxt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 6:08pm
speed-insights-remix ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 6:08pm
speed-insights-sveltekit ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 6:08pm
speed-insights-vue ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 6:08pm
feugy commented 3 days ago

Thanks a million! This shouldn't be needed thanks to #88