Skip to content

fix: allow npm run dev without Auth0 env vars#49

Merged
IvanKiral merged 1 commit intomainfrom
fix_kickstart_dev_without_auth0_config
Apr 30, 2026
Merged

fix: allow npm run dev without Auth0 env vars#49
IvanKiral merged 1 commit intomainfrom
fix_kickstart_dev_without_auth0_config

Conversation

@IvanKiral
Copy link
Copy Markdown
Contributor

Move Auth0 env reads from AuthProviderWithRedirect into main.tsx and pass domain, clientId, redirectUri as props. When any of VITE_AUTH_DOMAIN / VITE_AUTH_CLIENT_ID / VITE_AUTH_REDIRECT_URL is missing, the /:envId route is no longer registered and a console warning names the missing vars, so the dev server can boot without a populated .env.

Motivation

Which issue does this fix? Fixes #issue number

If no issue exists, what is the fix or new feature? Were there any reasons to fix/implement things that are not obvious?

Checklist

  • Code follows coding conventions held in this repo
  • Automated tests have been added
  • Tests are passing
  • Docs have been updated (if applicable)
  • Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

If manual testing is required, what are the steps?

Move Auth0 env reads from AuthProviderWithRedirect into main.tsx and pass
domain, clientId, redirectUri as props. When any of VITE_AUTH_DOMAIN /
VITE_AUTH_CLIENT_ID / VITE_AUTH_REDIRECT_URL is missing, the /:envId
route is no longer registered and a console warning names the missing
vars, so the dev server can boot without a populated .env.
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
kickstart Ready Ready Preview, Comment Apr 30, 2026 0:08am
kickstart-qa Ready Ready Preview, Comment Apr 30, 2026 0:08am

Request Review

@IvanKiral IvanKiral merged commit 97e686a into main Apr 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants