search

midday-ai / v1

An open-source starter kit based on Midday.
https://v1.run
MIT License
3.18k stars 285 forks source link

Vercel deploy fails because of sentry's confusing instructions #46

Open absolutegravitas opened 2 months ago

absolutegravitas commented 2 months ago

See https://github.com/getsentry/sentry-cli/issues/2137#issuecomment-2335192544

v1.run vercel deploy doesn't build and deploy onto vercel out of the box because of this confusing and inactionable message from sentry. I've attempted package updates on @sentry/nextjs to no effect.

Also general feedback for midday crew -- imo monitoring / logging env vars shouldnt be mandatory for example / template repos. Conditional checks should be added to test for null envs and bypass that feature in the v1 template.

pontusab commented 2 months ago

Thanks for your feedback! We have a nice change coming: https://github.com/midday-ai/v1/pull/45 where you can init a bare project (nextjs + supabase) and a full example with sentry etc, or you can add services as you want with pre-made implementaitons + envs

jaaneh commented 2 months ago

For anyone looking at this in the future, the SENTRY_AUTH_TOKEN env var needs to be a token you create on this page. Once that's set up, re-deploy the app, and that should fix the "Invalid token" error.